Skip to content

OKTA-939653 - Updating mushtache file#1635

Merged
prachi-okta merged 12 commits intomasterfrom
OKTA-939653-previewSAMLmetadataForApplication-issue
Jul 3, 2025
Merged

OKTA-939653 - Updating mushtache file#1635
prachi-okta merged 12 commits intomasterfrom
OKTA-939653-previewSAMLmetadataForApplication-issue

Conversation

@prachi-okta
Copy link
Copy Markdown
Contributor

@prachi-okta prachi-okta commented Jun 26, 2025

Issue(s)

https://oktainc.atlassian.net/browse/OKTA-939653

Description

This PR fixes the 400 Bad Request (E0000019) error encountered when invoking ApplicationSsoApi.previewSAMLmetadataForApplication(...).

Problem:
The endpoint /api/v1/apps/{appId}/sso/saml/metadata responds with text/xml, but the SDK was sending Accept: application/json.

This mismatch caused the API to return:

"errorSummary": "Bad request. Accept and/or Content-Type headers likely do not match supported values."

Fix:

  • Updated the SDK logic (in either the mustache template or ApiClient.java) to dynamically set the Accept header to text/xml for this endpoint.
  • Ensured the method’s return type remains String to safely handle raw XML without deserialization errors.
  • Avoided parsing or deserializing text/xml responses through Jackson.

Category

  • Bugfix
  • Enhancement
  • New Feature
  • Library Upgrade
  • Configuration Change
  • Versioning Change
  • Unit or Integration Test(s)
  • Documentation

Signoff

  • I have submitted a CLA for this PR
  • Each commit message explains what the commit does
  • I have updated documentation to explain what my PR does
  • My code is covered by tests if required
  • I did not edit any automatically generated files

@prachi-okta prachi-okta changed the title updating mushtache file OKTA-939653 - Updating mushtache file Jul 2, 2025
@prachi-okta prachi-okta merged commit 44a6c26 into master Jul 3, 2025
8 checks passed
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.

2 participants