Skip to content

fix swagger page to use the current API context#397

Merged
adamkorynta merged 1 commit intomainfrom
bugfix/swagger_try_it_out
Apr 1, 2025
Merged

fix swagger page to use the current API context#397
adamkorynta merged 1 commit intomainfrom
bugfix/swagger_try_it_out

Conversation

@adamkorynta
Copy link
Contributor

Problem Description

The Swagger try-it-out feature does not work if the REST API context is not the root.

Solution

Add the API context to the swagger servers list.

how you tested the change

Tested running the embedded Tomcat app.

Where the following done:

  • Tests. Check all that apply:
    • Unit tests created or modified that run during ant test.
    • Integration tests created or modified that run during integration testing
      (Formerly called regression tests.)
    • Test procedure descriptions for manual testing
  • Was relevant documentation updated?
  • Were relevant config element (e.g. XML data) updated as appropriate

@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 1, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 30%)

See analysis details on SonarQube Cloud

@adamkorynta adamkorynta merged commit 662035a into main Apr 1, 2025
8 of 9 checks passed
@adamkorynta adamkorynta deleted the bugfix/swagger_try_it_out branch April 1, 2025 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants