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

Deprecate --insecure-addr #763

Closed
tsandall opened this issue May 30, 2018 · 2 comments
Closed

Deprecate --insecure-addr #763

tsandall opened this issue May 30, 2018 · 2 comments
Assignees

Comments

@tsandall
Copy link
Member

With the recent changes to --addr (e.g., #762), we don't need --insecure-addr anymore. The goal will be to remove --insecure-addr by v0.11.

Work required:

  • Add support for multiple --addr values (v0.9)
  • Update docs to remove references to --insecure-addr and replace with --addr (v0.9)
  • Add warning message on startup if --insecure-addr is used (v0.9)
  • Remove --insecure-addr completely (v0.11)

/cc @JAORMX

@BenderScript
Copy link
Contributor

Let me take a stab at this one

tsandall pushed a commit that referenced this issue Oct 15, 2018
Related to #763. In the future, we can remove the flag entirely.

Signed-off-by: repenno <rapenno@gmail.com>
@tsandall
Copy link
Member Author

Changes have been merged. We can target removal for the release after this one.

@tsandall tsandall self-assigned this Jun 27, 2019
tsandall added a commit to tsandall/opa that referenced this issue Jul 26, 2019
--insecure-addr was deprecated in in Oct 2018. It's time to say
  goodbye.

Fixes open-policy-agent#763

Signed-off-by: Torin Sandall <torinsandall@gmail.com>
tsandall added a commit to tsandall/opa that referenced this issue Oct 21, 2020
--insecure-addr was deprecated in in Oct 2018. It's time to say
  goodbye.

Fixes open-policy-agent#763

Signed-off-by: Torin Sandall <torinsandall@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants