Skip to content

feat: allow bypassing current CORS magic based on template config #18706

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
+1,149 −108
Open

feat: allow bypassing current CORS magic based on template config #18706

Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
94e423b
wip
cstyan Jul 3, 2025
b2e9c6d
wip 2
cstyan Jul 3, 2025
5b2b80e
fix broken build and tests
cstyan Jul 7, 2025
1af9d2f
rename sql files
cstyan Jul 7, 2025
6c136cf
fix TS files for tests/linting
cstyan Jul 7, 2025
8b21d0c
wip proxy changes, still broken
cstyan Jul 8, 2025
20c3ffd
commit cors package
cstyan Jul 8, 2025
20dbf97
port test
cstyan Jul 8, 2025
432aff6
this should fix most tests, thanks blink
cstyan Jul 8, 2025
e75fb3b
Merge branch 'main' into callum-cors
cstyan Jul 9, 2025
5b5e766
move cors mw after app handle subdomain
cstyan Jul 9, 2025
039240b
Merge branch 'main' into callum-cors
cstyan Jul 9, 2025
8251d8e
merge missed removing dbmem
cstyan Jul 9, 2025
22c0711
fix leftover merge conflict
cstyan Jul 9, 2025
a5ad58b
fix linting
cstyan Jul 9, 2025
e6afe73
address review comments related to valid/validate functions
cstyan Jul 29, 2025
9aa5bdb
Merge branch 'main' into callum-cors
cstyan Jul 29, 2025
bc41ed4
move cors DB migration and re-make gen
cstyan Jul 29, 2025
6a62419
fix mistaken change in dbgen
cstyan Jul 29, 2025
fa93b9b
fix lint erro
cstyan Jul 29, 2025
4d498c6
somehow make gen doesn't fix this file?
cstyan Jul 29, 2025
a41dbcc
again, make gen doesn't seem to fix the required formatting here
cstyan Jul 29, 2025
92b6e20
address more review feedback
cstyan Jul 29, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix lint erro 
Signed-off-by: Callum Styan <callumstyan@gmail.com>
  • Loading branch information
@cstyan
cstyan committed Jul 29, 2025
commit fa93b9b0a86b1d55c262d1e7df8d28a80a4549de
6 changes: 4 additions & 2 deletions coderd/templates.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,8 @@ func (api *API) postTemplateByOrganization(rw http.ResponseWriter, r *http.Reque
val = database.CorsBehavior(*createTemplate.CORSBehavior)
}
if !val.Valid() {
validErrs = append(validErrs, codersdk.ValidationError{Field: "cors_behavior",
validErrs = append(validErrs, codersdk.ValidationError{
Field: "cors_behavior",
Detail: fmt.Sprintf("Invalid CORS behavior %q. Must be one of [%s]", *createTemplate.CORSBehavior, strings.Join(slice.ToStrings(database.AllCorsBehaviorValues()), ", ")),
})
} else {
Expand Down Expand Up @@ -746,7 +747,8 @@ func (api *API) patchTemplateMeta(rw http.ResponseWriter, r *http.Request) {
if req.CORSBehavior != nil && *req.CORSBehavior != "" {
val := database.CorsBehavior(*req.CORSBehavior)
if !val.Valid() {
validErrs = append(validErrs, codersdk.ValidationError{Field: "cors_behavior",
validErrs = append(validErrs, codersdk.ValidationError{
Field: "cors_behavior",
Detail: fmt.Sprintf("Invalid CORS behavior %q. Must be one of [%s]", *req.CORSBehavior, strings.Join(slice.ToStrings(database.AllCorsBehaviorValues()), ", ")),
})
} else {
Expand Down
Loading
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy