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

[BUG] fix type of payload in MessageExample #560

Closed
2 tasks done
Pakisan opened this issue Dec 30, 2024 · 1 comment · Fixed by #561
Closed
2 tasks done

[BUG] fix type of payload in MessageExample #560

Pakisan opened this issue Dec 30, 2024 · 1 comment · Fixed by #561
Assignees
Labels
bug Something isn't working released

Comments

@Pakisan
Copy link
Member

Pakisan commented Dec 30, 2024

Describe the bug.

Wrong definition of payload property

"headers": {
"type": "object",
"description": "Example of the application headers. It can be of any type."
},
"payload": {
"description": "Example of the message payload. It can be of any type."
}

Expected behavior

MessageExample.payload must be defined as any type

{
    "headers": {
      "type":  "object",
      "description": "Example of the application headers. It can be of any type."
    },
    "payload": {
      "description": "Example of the message payload. It can be of any type."
    }
}

See:

👀 Have you checked for similar open issues?

  • I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

Are you willing to work on this issue ?

Yes I am willing to submit a PR!

@Pakisan Pakisan added the bug Something isn't working label Dec 30, 2024
@Pakisan Pakisan self-assigned this Dec 30, 2024
Pakisan added a commit to Pakisan/spec-json-schemas that referenced this issue Dec 30, 2024
@Pakisan Pakisan changed the title [BUG] fix type of payload and headers in MessageExample [BUG] fix type of payload in MessageExample Jan 9, 2025
Pakisan added a commit that referenced this issue Jan 27, 2025
* fix: fix type of payload and headers in MessageExample

#560

* fix: wrong type of payload and headers in MessageExample

Rollback headers to Map[string, any]

* fix: headers description in MessageExample

Update headers description. Clarify that headers must be a map of key-value pairs

* Update definitions/3.0.0/messageExampleObject.json

Co-authored-by: Fran Méndez <[email protected]>

---------

Co-authored-by: Fran Méndez <[email protected]>
@asyncapi-bot
Copy link
Contributor

🎉 This issue has been resolved in version 6.8.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants