Skip to content

JWT: The XML input and output aren't supported#217

Merged
konradoboza merged 5 commits into5.0from
fix-jwt-doc
May 6, 2026
Merged

JWT: The XML input and output aren't supported#217
konradoboza merged 5 commits into5.0from
fix-jwt-doc

Conversation

@adriendupuis
Copy link
Copy Markdown
Contributor

@adriendupuis adriendupuis commented Mar 26, 2026

🎫 Issue IBX-10444

Related PRs:

Description:

For QA:

On an instance, look at REST API doc at /api/ibexa/v2/doc#/User%20Token/api_usertokenjwt_post, and eventually click the "Try it out" button.

Before the fix:

  • There are mandatory Accept and Content-Type parameter fields that are ignored.
  • You can choose between XML and JSON requests and responses.
  • If you test with an XML input, you get an error.
  • If you test with a XML reponse, you get a JSON response anyway.

After the fix:

image

Documentation:

REST API Reference should be automatically fixed on next commerce-skeleton release tagging.

@adriendupuis adriendupuis changed the title JWT: The XML input and output aren't supported since #101 JWT: The XML input and output aren't supported Mar 26, 2026
Avoid the default "The new JWT resource" description
@sonarqubecloud
Copy link
Copy Markdown

@ibexa-yuna ibexa-yuna deleted the branch 5.0 April 29, 2026 08:23
@ibexa-yuna ibexa-yuna closed this Apr 29, 2026
@mnocon mnocon reopened this Apr 29, 2026
@sonarqubecloud
Copy link
Copy Markdown

@mnocon mnocon changed the base branch from main to 5.0 April 29, 2026 09:21
Copy link
Copy Markdown

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA approved on Ibexa DXP 5.0.

@konradoboza konradoboza merged commit 922fff9 into 5.0 May 6, 2026
23 of 24 checks passed
@konradoboza konradoboza deleted the fix-jwt-doc branch May 6, 2026 12:57
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.

9 participants