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

Consul support uses a deprecated token query parameter #22677

Open
mraible opened this issue Jun 26, 2023 · 8 comments
Open

Consul support uses a deprecated token query parameter #22677

mraible opened this issue Jun 26, 2023 · 8 comments
Assignees
Labels
$$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ theme: microservice $100 https://www.jhipster.tech/bug-bounties/

Comments

@mraible
Copy link
Contributor

mraible commented Jun 26, 2023

Overview of the issue

I created a microservices architecture using my Micro Frontends demo script today and noticed there are a lot of deprecation warnings from Consul:

[WARN]  agent.http: This request used the token query parameter which is deprecated and will be removed in Consul 1.17: 
logUrl=/v1/catalog/services?token=<hidden>
Motivation for or Use Case

It'd be nice if there weren't deprecation warnings in log files.

Reproduce the error

Following my demo script and you'll see the warnings after you start the full stack with docker compose up.

JHipster Version(s)

main branch as of June 26, 2023.

@deepu105
Copy link
Member

There is also no OIDC support since its an enterprise feature in Consul

@mraible
Copy link
Contributor Author

mraible commented Jun 29, 2023

It says the token query parameter is deprecated and will be removed in Consul 1.17. Should we be concerned?

@mraible mraible added $100 https://www.jhipster.tech/bug-bounties/ $$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ and removed area: triage labels Jul 8, 2023
@mraible
Copy link
Contributor Author

mraible commented Jul 8, 2023

I added a bug bounty to get this fixed.

@Survival1sm
Copy link

This appears to be an issue with the unmaintained consul-api library used by spring-cloud-consul for http requests.

Issue on spring-cloud-consul
Issue on consul-api
Issue for Spring Cloud Consul to migrate away from consul-api: spring-cloud-consul

@BrayanMnz
Copy link
Contributor

Should this be worked on or already done? @mraible

@mraible
Copy link
Contributor Author

mraible commented Aug 14, 2024

This is still an issue as far as I know.

@BrayanMnz
Copy link
Contributor

Assign this to me, I will take a look at it @mraible

@mraible
Copy link
Contributor Author

mraible commented Aug 27, 2024

@BrayanMnz Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
$$ bug-bounty $$ https://www.jhipster.tech/bug-bounties/ theme: microservice $100 https://www.jhipster.tech/bug-bounties/
Projects
None yet
Development

No branches or pull requests

4 participants