Skip to content
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

feat: allow unencoded userinfo as per SIP002 #260

Merged
merged 5 commits into from
Jul 16, 2024
Merged

feat: allow unencoded userinfo as per SIP002 #260

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
adf5a6a
feat: allow unencoded `userinfo` as per SIP002
sbruens Jul 15, 2024
30002e1
Move shadowsocks config tests to `shadowsocks_test.go` and clean up.
sbruens Jul 15, 2024
024d870
Review comment.
sbruens Jul 15, 2024
435f8a3
Add test case for invalid cipher info (e.g. missin colon).
sbruens Jul 15, 2024
3c282b9
Remove unnecessary prefix in test.
sbruens Jul 15, 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
61 changes: 0 additions & 61 deletions x/config/config_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@
package config

import (
"encoding/base64"
"testing"

"github.com/stretchr/testify/require"
Expand Down Expand Up @@ -55,69 +54,9 @@ func TestSanitizeConfig(t *testing.T) {
require.Error(t, err)
}

func TestShowsocksLagacyBase64URL(t *testing.T) {
encoded := base64.URLEncoding.WithPadding(base64.NoPadding).EncodeToString([]byte("aes-256-gcm:[email protected]:1234?prefix=HTTP%2F1.1%20"))
urls, err := parseConfig("ss://" + string(encoded) + "#outline-123")
require.NoError(t, err)
require.Equal(t, 1, len(urls))
config, err := parseShadowsocksLegacyBase64URL(urls[0])
require.Equal(t, "example.com:1234", config.serverAddress)
require.Equal(t, "HTTP/1.1 ", string(config.prefix))
require.NoError(t, err)
}

func TestParseShadowsocksURL(t *testing.T) {
encoded := base64.URLEncoding.WithPadding(base64.NoPadding).EncodeToString([]byte("aes-256-gcm:[email protected]:1234?prefix=HTTP%2F1.1%20"))
urls, err := parseConfig("ss://" + string(encoded) + "#outline-123")
require.NoError(t, err)
require.Equal(t, 1, len(urls))
config, err := parseShadowsocksURL(urls[0])
require.Equal(t, "example.com:1234", config.serverAddress)
require.Equal(t, "HTTP/1.1 ", string(config.prefix))
require.NoError(t, err)

encoded = base64.URLEncoding.WithPadding(base64.NoPadding).EncodeToString([]byte("aes-256-gcm:1234567"))
urls, err = parseConfig("ss://" + string(encoded) + "@example.com:1234?prefix=HTTP%2F1.1%20" + "#outline-123")
require.NoError(t, err)
require.Equal(t, 1, len(urls))
config, err = parseShadowsocksURL(urls[0])
require.Equal(t, "example.com:1234", config.serverAddress)
require.Equal(t, "HTTP/1.1 ", string(config.prefix))
require.NoError(t, err)
}

func TestSocks5URLSanitization(t *testing.T) {
configString := "socks5://myuser:[email protected]:1080"
sanitizedConfig, err := SanitizeConfig(configString)
require.NoError(t, err)
require.Equal(t, "socks5://[email protected]:1080", sanitizedConfig)
}

func TestParseShadowsocksSIP002URLUnsuccessful(t *testing.T) {
encoded := base64.URLEncoding.WithPadding(base64.NoPadding).EncodeToString([]byte("aes-256-gcm:[email protected]:1234?prefix=HTTP%2F1.1%20"))
urls, err := parseConfig("ss://" + string(encoded) + "#outline-123")
require.NoError(t, err)
require.Equal(t, 1, len(urls))
_, err = parseShadowsocksSIP002URL(urls[0])
require.Error(t, err)
}

func TestParseShadowsocksSIP002URLUnsupportedCypher(t *testing.T) {
configString := "ss://Y2hhY2hhMjAtaWV0Zi1wb2x5MTMwnTpLeTUyN2duU3FEVFB3R0JpQ1RxUnlT@example.com:1234?prefix=HTTP%2F1.1%20"
urls, err := parseConfig(configString)
require.NoError(t, err)
require.Equal(t, 1, len(urls))
_, err = parseShadowsocksSIP002URL(urls[0])
require.Error(t, err)
}

func TestParseShadowsocksSIP002URLSuccessful(t *testing.T) {
configString := "ss://Y2hhY2hhMjAtaWV0Zi1wb2x5MTMwNTpLeTUyN2duU3FEVFB3R0JpQ1RxUnlT@example.com:1234?prefix=HTTP%2F1.1%20"
urls, err := parseConfig(configString)
require.NoError(t, err)
require.Equal(t, 1, len(urls))
config, err := parseShadowsocksSIP002URL(urls[0])
require.NoError(t, err)
require.Equal(t, "example.com:1234", config.serverAddress)
require.Equal(t, "HTTP/1.1 ", string(config.prefix))
}
14 changes: 10 additions & 4 deletions x/config/shadowsocks.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,11 +144,17 @@ func parseShadowsocksSIP002URL(url *url.URL) (*shadowsocksConfig, error) {
return nil, errors.New("host not specified")
}
config.serverAddress = url.Host
cipherInfoBytes, err := base64.URLEncoding.WithPadding(base64.NoPadding).DecodeString(url.User.String())
if err != nil {
return nil, fmt.Errorf("failed to decode cipher info [%v]: %w", url.User.String(), err)
userInfo := url.User.String()
// Cipher info can be optionally encoded with Base64URL.
encoding := base64.URLEncoding.WithPadding(base64.NoPadding)
decodedUserInfo, err := encoding.DecodeString(userInfo)
var cipherInfo string
if err == nil {
cipherInfo = string(decodedUserInfo)
} else {
cipherInfo = userInfo
}
cipherName, secret, found := strings.Cut(string(cipherInfoBytes), ":")
cipherName, secret, found := strings.Cut(cipherInfo, ":")
if !found {
return nil, errors.New("invalid cipher info: no ':' separator")
}
Expand Down
85 changes: 85 additions & 0 deletions x/config/shadowsocks_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
package config

import (
"encoding/base64"
"net/url"
"testing"

Expand All @@ -36,3 +37,87 @@ func Test_sanitizeShadowsocksURL_withPrefix(t *testing.T) {
require.NoError(t, err)
require.Equal(t, "ss://[email protected]:8888?prefix=foo", sanitized)
}

func TestParseShadowsocksURLFullyEncoded(t *testing.T) {
encoded := base64.URLEncoding.WithPadding(base64.NoPadding).EncodeToString([]byte("aes-256-gcm:[email protected]:1234?prefix=HTTP%2F1.1%20"))
urls, err := parseConfig("ss://" + string(encoded) + "#outline-123")
require.NoError(t, err)
require.Equal(t, 1, len(urls))

config, err := parseShadowsocksURL(urls[0])

require.NoError(t, err)
require.Equal(t, "example.com:1234", config.serverAddress)
require.Equal(t, "HTTP/1.1 ", string(config.prefix))
}

func TestParseShadowsocksURLUserInfoEncoded(t *testing.T) {
encoded := base64.URLEncoding.WithPadding(base64.NoPadding).EncodeToString([]byte("aes-256-gcm:1234567"))
urls, err := parseConfig("ss://" + string(encoded) + "@example.com:1234?prefix=HTTP%2F1.1%20" + "#outline-123")
require.NoError(t, err)
require.Equal(t, 1, len(urls))

config, err := parseShadowsocksURL(urls[0])

require.NoError(t, err)
require.Equal(t, "example.com:1234", config.serverAddress)
require.Equal(t, "HTTP/1.1 ", string(config.prefix))
}

func TestParseShadowsocksURLNoEncoding(t *testing.T) {
configString := "ss://aes-256-gcm:[email protected]:1234"
urls, err := parseConfig(configString)
require.NoError(t, err)
require.Equal(t, 1, len(urls))

config, err := parseShadowsocksURL(urls[0])

require.NoError(t, err)
require.Equal(t, "example.com:1234", config.serverAddress)
}

func TestParseShadowsocksURLInvalidCipherInfoFails(t *testing.T) {
configString := "ss://[email protected]:1234"
urls, err := parseConfig(configString)
require.NoError(t, err)
require.Equal(t, 1, len(urls))

_, err = parseShadowsocksURL(urls[0])

require.Error(t, err)
}

func TestParseShadowsocksURLUnsupportedCypherFails(t *testing.T) {
configString := "ss://Y2hhY2hhMjAtaWV0Zi1wb2x5MTMwnTpLeTUyN2duU3FEVFB3R0JpQ1RxUnlT@example.com:1234"
urls, err := parseConfig(configString)
require.NoError(t, err)
require.Equal(t, 1, len(urls))

_, err = parseShadowsocksURL(urls[0])

require.Error(t, err)
}

func TestParseShadowsocksLegacyBase64URL(t *testing.T) {
encoded := base64.URLEncoding.WithPadding(base64.NoPadding).EncodeToString([]byte("aes-256-gcm:[email protected]:1234?prefix=HTTP%2F1.1%20"))
urls, err := parseConfig("ss://" + string(encoded) + "#outline-123")
require.NoError(t, err)
require.Equal(t, 1, len(urls))

config, err := parseShadowsocksLegacyBase64URL(urls[0])

require.NoError(t, err)
require.Equal(t, "example.com:1234", config.serverAddress)
require.Equal(t, "HTTP/1.1 ", string(config.prefix))
}

func TestParseShadowsocksSIP002URLUnsuccessful(t *testing.T) {
encoded := base64.URLEncoding.WithPadding(base64.NoPadding).EncodeToString([]byte("aes-256-gcm:[email protected]:1234?prefix=HTTP%2F1.1%20"))
urls, err := parseConfig("ss://" + string(encoded) + "#outline-123")
require.NoError(t, err)
require.Equal(t, 1, len(urls))

_, err = parseShadowsocksSIP002URL(urls[0])

require.Error(t, err)
}
Loading