Skip to content

Comments

Allow custom stream element attributes (+ equivalent for WS)#281

Merged
pawlooss1 merged 1 commit intomasterfrom
customizable-stream-element
Sep 3, 2025
Merged

Allow custom stream element attributes (+ equivalent for WS)#281
pawlooss1 merged 1 commit intomasterfrom
customizable-stream-element

Conversation

@chrzaszcz
Copy link
Member

@chrzaszcz chrzaszcz commented Sep 1, 2025

Allow custom stream element attributes.

These attributes apply to:

  • <stream:stream> tag for TCP
  • <open> and <close> elements for Websockets

Additionally, facilitate testing error conditions by allowing the undefined value for the server JID and attributes.
Such values result in the attributes being removed from the respective element.

@chrzaszcz chrzaszcz marked this pull request as draft September 1, 2025 14:31
@chrzaszcz chrzaszcz force-pushed the customizable-stream-element branch 3 times, most recently from c9023b0 to 87906b6 Compare September 3, 2025 07:26
These attributes apply to:
  - <stream:stream> tag for TCP
  - <open> and <close> elements for Websockets

Additionally, facilitate testing error conditions by allowing
the 'undefined' value for the server and attributes.
Such values result in the attributes being removed from the respective
element.
@chrzaszcz chrzaszcz force-pushed the customizable-stream-element branch from 87906b6 to f611708 Compare September 3, 2025 09:41
@chrzaszcz chrzaszcz marked this pull request as ready for review September 3, 2025 14:26
Copy link
Member

@pawlooss1 pawlooss1 left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@pawlooss1 pawlooss1 merged commit a50c455 into master Sep 3, 2025
3 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