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: Correct API endpoint mismatch in CredentialController #50

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cinari4
Copy link

@cinari4 cinari4 commented Aug 6, 2023

What is this PR for?

Fixing API Endpoint Mismatch Issue in CredentialController

Overview or reasons

I have identified an issue in the CredentialController on the server.
Currently, the endpoint for receiving requests is set as /fido2/credentials/{id}, but I noticed that when constructing the API address on the rpserver, it is being called as /fido2/credentials{id}.
As a result, the credentials API cannot be successfully invoked.

Tasks

When calling the credentials API, you need to configure the address of that API in rpserver, pathSegment function when calling the credentials API.

Result

Now able to call the server's credential/{id} api from rpserver as normal.

@CLAassistant
Copy link

CLAassistant commented Aug 6, 2023

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants