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

[1º] - Hotfix/v1.3.1-discovery-service: Fixed timeout issue with inexistent BPN and EDC Discoveries listed in the "Discovery Service" #147

Merged

Conversation

matbmoser
Copy link
Contributor

@matbmoser matbmoser commented Nov 8, 2023

Why we create this PR?

There is the need to fix a bug related to the BPN and EDC Discovery when the APIs configured there dont exist.

What we want to achieve with this PR?

We want to solve the problem by introducing a configurable timeout in the calls.

What is new?

Added

  • Added timeout for the BPN Discovery and EDC Discovery APIs returned in the Discovery Service API.
  • Created a new doPost function in the HttpUtilsthat allows the usage of timeouts in API calls.
  • Create a new timeout function in ThreadUtils to be applied to the call of functions.

@matbmoser matbmoser added bug Something isn't working backend Issue or PR connected to the backend application labels Nov 8, 2023
@matbmoser matbmoser added this to the v1.3.1 milestone Nov 8, 2023
@matbmoser matbmoser force-pushed the hotfix/v1.3.1-discovery-services branch 3 times, most recently from ed40a84 to 400aa6e Compare November 8, 2023 16:58
@matbmoser matbmoser force-pushed the hotfix/v1.3.1-discovery-services branch from 400aa6e to 42b4c2c Compare November 8, 2023 17:03
@matbmoser matbmoser merged commit f492e49 into eclipse-tractusx:main Nov 8, 2023
5 checks passed
@matbmoser matbmoser deleted the hotfix/v1.3.1-discovery-services branch November 8, 2023 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Issue or PR connected to the backend application bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant