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

Include urlFragment into URLs reported by network events #2077

Closed
OrKoN opened this issue Mar 27, 2024 · 0 comments · Fixed by #2079
Closed

Include urlFragment into URLs reported by network events #2077

OrKoN opened this issue Mar 27, 2024 · 0 comments · Fixed by #2079
Assignees
@Lightning00Blade
Labels
network

Comments

@OrKoN
Copy link
Collaborator

OrKoN commented Mar 27, 2024

According to the spec, the urls should include the fragment but CDP does not include in the url attribute. Instead it is included as urlFragment on the Network request.
@OrKoN OrKoN added the network label Mar 27, 2024
@Lightning00Blade Lightning00Blade mentioned this issue Mar 27, 2024
Merged
Lightning00Blade added a commit that referenced this issue Mar 27, 2024
@Lightning00Blade
fix: add fragment to url (#2079)
d416b6c 
Closes #2077

---------

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Lightning00Blade <Lightning00Blade@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Lightning00Blade Lightning00Blade

Labels
network
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: add fragment to url GoogleChromeLabs/chromium-bidi
2 participants
@OrKoN @Lightning00Blade