Skip to content

Commit

Permalink
Merge pull request #727 from v3rm0n/hash-url
Browse files Browse the repository at this point in the history
Allow hash in URL
  • Loading branch information
Mark Wolfe authored Oct 20, 2021
2 parents 050bd47 + bb718bf commit 8c1b6fe
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion pkg/cfg/cfg.go
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ func (cm *ConfigManager) SaveIDPAccount(idpAccountName string, account *IDPAccou
// LoadIDPAccount load the idp account and default to an empty one if it doesn't exist
func (cm *ConfigManager) LoadIDPAccount(idpAccountName string) (*IDPAccount, error) {

cfg, err := ini.LoadSources(ini.LoadOptions{Loose: true}, cm.configPath)
cfg, err := ini.LoadSources(ini.LoadOptions{Loose: true, SpaceBeforeInlineComment: true}, cm.configPath)
if err != nil {
return nil, errors.Wrap(err, "Unable to load configuration file")
}
Expand Down
2 changes: 1 addition & 1 deletion pkg/cfg/cfg_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ func TestNewConfigManagerLoad(t *testing.T) {
require.Nil(t, err)
require.Equal(t, &IDPAccount{
Name: "test123",
URL: "https://id.whatever.com",
URL: "https://id.whatever.com/#/hash",
Username: "[email protected]",
Provider: "keycloak",
MFA: "sms",
Expand Down
3 changes: 1 addition & 2 deletions pkg/cfg/example/saml2aws.ini
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ mfa = totp
username = [email protected]
provider = keycloak
mfa = sms
url = https://id.whatever.com
url = https://id.whatever.com/#/hash
skip_verify = false
timeout = 0

0 comments on commit 8c1b6fe

Please sign in to comment.