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.

Sign up for GitHub

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

Jump to bottom

Add origin property to MessageEvent #3

Closed
clue opened this issue Apr 12, 2019 · 2 comments
Closed

Add origin property to MessageEvent #3

clue opened this issue Apr 12, 2019 · 2 comments
Labels
help wanted Extra attention is needed new feature New feature or request

Comments

@clue
Copy link
Owner

clue commented Apr 12, 2019

The MessageEvent should have an origin property that references the URL of the EventSource object (after following any redirects): https://html.spec.whatwg.org/multipage/comms.html#dom-messageevent-origin

Depends on clue/reactphp-buzz#129 in order to be able to access the last effective URL.
@clue clue added new feature New feature or request help wanted Extra attention is needed labels Apr 12, 2019
@clue
Copy link
Owner Author

clue commented Apr 11, 2022

For the reference: This now depends on reactphp/http#388 after the HTTP client has been updated via #13.

@clue
Copy link
Owner Author

clue commented Apr 7, 2023

I still think that this feature makes perfect sense 👍 However, there are currently no immediate plans to build this from my end (no demand at the moment and more important outstanding issues currently), so I'm closing this for now.

We welcome contributions, reach out if you want to support this project 👍

@clue clue closed this as completed Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed new feature New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@clue