Skip to content

Conversation

@TimHess
Copy link
Member

@TimHess TimHess commented Feb 24, 2025

Description

Remove the override that forced mappings label responses as v2.
Also throw an exception if SBA url wasn't set rather than trying to connect first.

Quality checklist

  • Your code complies with our Coding Style.
  • You've updated unit and/or integration tests for your change, where applicable.
  • You've updated documentation for your change, where applicable.
    If your change affects other repositories, such as Documentation, Samples and/or MainSite, add linked PRs here.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.
  • You've added required license files and/or file headers (explaining where the code came from with proper attribution), where code is copied from StackOverflow, a blog, or OSS.

@TimHess TimHess requested a review from bart-vmware February 24, 2025 16:49
@TimHess
Copy link
Member Author

TimHess commented Feb 24, 2025

Tested with SBA 3.4.3 built from SteeltoeOSS/Dockerfiles#42
image

@TimHess TimHess added Component/Management Issues related to Steeltoe Management (actuators) ReleaseLine/4.x Identified as a feature/fix for the 4.x release line labels Feb 24, 2025
@TimHess TimHess force-pushed the mappings-content-type branch from 46bb0b5 to 9325dac Compare February 25, 2025 17:08
@sonarqubecloud
Copy link

@TimHess TimHess merged commit 7a19348 into main Feb 25, 2025
19 checks passed
@TimHess TimHess deleted the mappings-content-type branch February 25, 2025 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component/Management Issues related to Steeltoe Management (actuators) ReleaseLine/4.x Identified as a feature/fix for the 4.x release line

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants