Skip to content

feat: validate the cc when using city params #175

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
gandol wants to merge 10 commits into
base: master
Choose a base branch
from
+62 −6
Open

feat: validate the cc when using city params #175

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
1c5e752
feat: validate the cc when using city params
gika-d Feb 10, 2024
9e912b8
fix: add tmpVar
gika-d Feb 11, 2024
9db0227
feat: add test for validating cc
gika-d Feb 11, 2024
a0d6ca2
Update speedtest/server_test.go
gandol Feb 11, 2024
f08ae55
feat: add new param for config
gika-d Mar 2, 2024
65b9936
feat: if params --cc was passed use the cc
gika-d Mar 2, 2024
eccd4c7
Merge branch 'master' into fix/validate_cc
gandol Mar 2, 2024
89d46a9
update cc filter
r3inbowari Mar 2, 2024
c0d153f
fix: change to upper case
r3inbowari Mar 2, 2024
ae12748
remove disable tag
r3inbowari Mar 2, 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
9 changes: 9 additions & 0 deletions speedtest/server.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ type Server struct {
DLSpeed float64 `json:"dl_speed"`
ULSpeed float64 `json:"ul_speed"`
TestDuration TestDuration `json:"test_duration"`
CC string `json:"cc"`

Context *Speedtest `json:"-"`
}
Expand Down Expand Up @@ -254,6 +255,14 @@ func (s *Speedtest) FetchServerListContext(ctx context.Context) (Servers, error)
if err = decoder.Decode(&servers); err != nil {
return servers, err
}
if s.config.Location != nil {
for _, server := range servers {
if server.CC != strings.ToUpper(s.config.Location.CC) {
servers = append(servers[:0], servers[1:]...)
Comment thread
r3inbowari marked this conversation as resolved.
Outdated
}
}

}
case typeXMLPayload:
var list ServerList
// Decode xml
Expand Down