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
check string in body
  • Loading branch information
@f0ssel
f0ssel committed Apr 9, 2024
commit 3c9790e510cb3166a1298efef45258fba6532bfd
5 changes: 5 additions & 0 deletions coderd/workspaceapps/db_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -677,6 +677,11 @@ func Test_ResolveRequest(t *testing.T) {
})
// should parse as app and fail to find app "9090ss"
require.False(t, ok)
w := rw.Result()
_ = w.Body.Close()
b, err := io.ReadAll(w.Body)
require.NoError(t, err)
require.Contains(t, string(b), "404 - Application Not Found")
})

t.Run("SubdomainEndsInS", func(t *testing.T) {
Expand Down