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

cmd: Deprecate hydra connect and replace with per-command flags and environment variables #840

Closed
aeneasr opened this issue May 4, 2018 · 0 comments
Assignees
Labels
breaking change Changes behavior in a breaking manner. package/cli
Milestone

Comments

@aeneasr
Copy link
Member

aeneasr commented May 4, 2018

No description provided.

@aeneasr aeneasr added breaking change Changes behavior in a breaking manner. announcement package/cli labels May 4, 2018
@aeneasr aeneasr added this to the 1.0.0 milestone May 4, 2018
@aeneasr aeneasr self-assigned this May 4, 2018
aeneasr pushed a commit that referenced this issue May 4, 2018
This patch deprecates the `hydra connect` command as internal
access control has been removed from ORY Hydra and this command
no longer serves any purpose.

Instead, all commands are supplied with environment variables `HYDRA_URL`,
`OAUTH2_CLIENT_ID`, `OAUTH2_CLIENT_SECRET`, `OAUTH2_ACCESS_TOKEN`.

Please check out `hydra help <command>` for usage instructions. You
should also check out the upgrade guide for more detailed upgrade instructions.

This patch also renames some flags and command names which have been
documented in the upgrade guide.

Closes #841
Closes #840
aeneasr pushed a commit that referenced this issue May 4, 2018
This patch deprecates the `hydra connect` command as internal
access control has been removed from ORY Hydra and this command
no longer serves any purpose.

Instead, all commands are supplied with environment variables `HYDRA_URL`,
`OAUTH2_CLIENT_ID`, `OAUTH2_CLIENT_SECRET`, `OAUTH2_ACCESS_TOKEN`.

Please check out `hydra help <command>` for usage instructions. You
should also check out the upgrade guide for more detailed upgrade instructions.

This patch also renames some flags and command names which have been
documented in the upgrade guide.

Closes #841
Closes #840
@aeneasr aeneasr closed this as completed May 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Changes behavior in a breaking manner. package/cli
Projects
None yet
Development

No branches or pull requests

1 participant