Skip to content

Conversation

@ayeshLK
Copy link
Member

@ayeshLK ayeshLK commented Jun 2, 2025

Purpose

$subject

Comment on lines 16 to 22
This proposal introduces support for application-specific content types in the Ballerina `websubhub` library, enabling more flexible content
handling during WebSub content distribution. The [WebSub specification](https://www.w3.org/TR/websub/) allows a `topic` to declare any
`Content-Type` for its content and does not restrict it to common MIME types such as `application/json`, `application/xml`, or `text/plain`.

This enhancement focuses on supporting custom media types under the `application/*` top-level MIME type. These include structured and
vendor-specific types such as `application/secevent+jwt`, `application/vnd.example+json`, and others. By treating such types as first-class
citizens, Ballerina will be able to interoperate more effectively in real-world WebSub ecosystems that rely on non-standard content types.
Copy link
Member

Choose a reason for hiding this comment

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

Shall we remove the line breaks within a paragraph?

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed with the following commit:

ThisaruGuruge
ThisaruGuruge previously approved these changes Jun 2, 2025
Copy link
Member

@ThisaruGuruge ThisaruGuruge left a comment

Choose a reason for hiding this comment

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

Content LGTM

Co-authored-by: Thisaru Guruge <thisaru@wso2.com>
ThisaruGuruge
ThisaruGuruge previously approved these changes Jun 3, 2025
@ayeshLK ayeshLK dismissed sameerajayasoma’s stale review August 19, 2025 19:28

The merge-base changed after approval.

@ayeshLK ayeshLK dismissed sameerajayasoma’s stale review August 20, 2025 02:09

The merge-base changed after approval.

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.

6 participants