Skip to content

feat: add s suffix to use HTTPS for ports #12862

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

Merged
merged 10 commits into from
Apr 9, 2024
Merged

feat: add s suffix to use HTTPS for ports #12862

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
87ae0be
feat: add s suffix to use HTTPS for ports
f0ssel Apr 3, 2024
e8d0fae
add valudate tests
f0ssel Apr 5, 2024
3fe6787
add more tests
f0ssel Apr 5, 2024
0ab922a
reverse parse order
f0ssel Apr 8, 2024
b3d60a4
add tests
f0ssel Apr 8, 2024
755d1ef
fix vscode file
f0ssel Apr 8, 2024
816660c
let shared port protocol call shots for now
f0ssel Apr 8, 2024
f090196
lint
f0ssel Apr 8, 2024
5ee05b8
random lint
f0ssel Apr 8, 2024
3c9790e
check string in body
f0ssel Apr 9, 2024
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
let shared port protocol call shots for now
  • Loading branch information
@f0ssel
f0ssel committed Apr 8, 2024
commit 816660c4f0b22e2ee989da8e8798d81aeeb8515f
13 changes: 8 additions & 5 deletions coderd/workspaceapps/request.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -287,15 +287,14 @@ func (r Request) getDatabase(ctx context.Context, db database.Store) (*databaseR
// whether the app is a slug or a port and whether there are multiple agents
// in the workspace or not.
var (
agentNameOrID = r.AgentNameOrID
appURL string
appSharingLevel database.AppSharingLevel
agentNameOrID = r.AgentNameOrID
appURL string
appSharingLevel database.AppSharingLevel
// First check if it's a port-based URL with an optional "s" suffix for HTTPS.
potentialPortStr = strings.TrimSuffix(r.AppSlugOrPort, "s")
portUint, portUintErr = strconv.ParseUint(potentialPortStr, 10, 16)
)
if portUintErr == nil {
// If we fail to parse the port, see if it's a port with a trailing "s" for
// HTTPS.
protocol := "http"
if strings.HasSuffix(r.AppSlugOrPort, "s") {
protocol = "https"
Expand Down Expand Up @@ -350,6 +349,10 @@ func (r Request) getDatabase(ctx context.Context, db database.Store) (*databaseR
}
// No port share found, so we keep default to owner.
} else {
if ps.Protocol == database.PortShareProtocolHttps {
// Apply HTTPS protocol if specified.
appURL = fmt.Sprintf("https://127.0.0.1:%d", portUint)
}
appSharingLevel = ps.ShareLevel
}
} else {
Expand Down