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(registration): bpdm pool address for membership activation #518

Conversation

typecastcloud
Copy link
Contributor

@typecastcloud typecastcloud commented Feb 26, 2025

Description

Use configurable poolApiPath in BPDM pool address for application activation.

Why

Hardcoded path in portal backend does not follow intended behavior if Pool address is using any additional pathing other than v6.

Issue

Refs: eclipse-tractusx/portal-backend#1299

Checklist

  • I have performed a self-review of my changes
  • I have successfully tested my changes

Hardcoded path in portal backend does not follow intended behavior.

Refs: eclipse-tractusx/portal-backend#1299
Copy link
Contributor

@tfjanjua tfjanjua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change looks fine to me, we would need to remove v6/ part from backend as well so, waiting for fix from backend as well.

@tfjanjua tfjanjua self-assigned this Feb 27, 2025
Copy link
Contributor

@tfjanjua tfjanjua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the required backend change has also been done eclipse-tractusx/portal-backend#1300

@typecastcloud
I have tested your changes on my local and its working fine
I tried with 2 BPNs

BPNL000000000065
BPNL00000000015G

and received success response from the pool/v6/cx-memberships API

@typecastcloud typecastcloud merged commit 6bd1892 into release-candidate Feb 27, 2025
6 checks passed
@typecastcloud typecastcloud deleted the fix/bpdm-pool-address-for-membership-activation branch February 27, 2025 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: USER READY
Development

Successfully merging this pull request may close these issues.

5 participants