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.

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

Fix broken link #1372

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix broken link #1372

wants to merge 1 commit into from

Conversation

rhiroe
Copy link

@rhiroe rhiroe commented Dec 14, 2024

Background

The link that comes up when using the --provider flag is broken.

$ saml2aws login --provider=Ping
The --provider flag has been replaced with a new configure command. See https://github.com/versent/saml2aws/v2#adding-idp-accounts

Detail

Corrected https://github.com/versent/saml2aws/v2#adding-idp-accounts link to https://github.com/Versent/saml2aws#configuring-idp-accounts .

Additional info

The adding-idp-accounts document has been renamed to configuring-idp-accounts with this change.

7bbdacc#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5L71

Also consistent with what we have here.

-i, --provider=PROVIDER This flag is obsolete. See: https://github.com/Versent/saml2aws#configuring-idp-accounts

## Background

The link that comes up when using the `--provider` flag is broken.

```sh
$ saml2aws login --provider=Ping
The --provider flag has been replaced with a new configure command. See https://github.com/versent/saml2aws/v2#adding-idp-accounts
```

## Detail

Corrected https://github.com/versent/saml2aws/v2#adding-idp-accounts link to https://github.com/Versent/saml2aws#configuring-idp-accounts .

## Additional info

The adding-idp-accounts document has been renamed to configuring-idp-accounts with this change.

Versent@7bbdacc#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5L71

Also consistent with what we have here.

https://github.com/Versent/saml2aws/blob/64255ff7019612e60de7b5139206f1c514cf4374/README.md?plain=1#L212
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant