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

KNOX-2958 - Fixed API samples for certain services #814

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

smolnar82
Copy link
Contributor

What changes were proposed in this pull request?

Testing uncovered some API samples in the existing service definition files that needed to be fixed.

Additionally, a general improvement is implemented that adds the missing slash at the beginning of the path element if it was missing from the service definition sample.

How was this patch tested?

Added unit tests and confirmed that API samples are actually changed on the UI:
Screenshot 2023-10-31 at 10 26 44
Screenshot 2023-10-31 at 10 27 09
Screenshot 2023-10-31 at 10 27 21
To see the new-style API service dialog it's essential that you set the following property in gateway-site.xml:

    <property>
        <name>knox.homepage.api.services.view.version</name>
        <value>v2</value>
    </property>

Additionally, a general improvement is implemented that adds the missing slash at the beginning of the path element if it was missing from the service definition sample.
@smolnar82 smolnar82 merged commit 6ec81a0 into apache:master Oct 31, 2023
2 checks passed
@smolnar82 smolnar82 deleted the KNOX-2958 branch October 31, 2023 11:46
stoty pushed a commit to stoty/knox that referenced this pull request May 14, 2024
…e#814)

Additionally, a general improvement is implemented that adds the missing slash at the beginning of the path element if it was missing from the service definition sample.

Change-Id: I6803c7b72c38bc935bcf85b9e871b7e1c6604b18
stoty pushed a commit to stoty/knox that referenced this pull request May 14, 2024
* changes:
  CDPD-61199: Updated public sample API document links to match versions in CDP
  CDPD-61196, KNOX-2958 - Fixed API samples for certain services (apache#814)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants