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]: Schemes not respected. #41548
Comments
It seems the streaming behavior is the opposite of what's expected. I think flipping the stream property for each scheme could fix it : true for enable, false for disable in main.js |
Another thing; when I set |
@jmercouris see #41032 (which you were tagged in i think?) tl;dr it's a misleading privilege that was implemented confusingly, and should probably be removed instead. |
I agree, it should probably be removed. However, there is an important point above within this issue that has not been addressed.
Please don't close issues unless the problems have been addressed. There is still a very deep bug that dispatch won't happen for 'standard' schemes when the URL appears to be malformed. |
Perhaps I should open up a new issue for this. |
@jmercouris i hear you but the initial issue is a duplicate of an open one - i folded it into that issue which remains open. the second comment you made: is a different issue. I can re-open this issue if you change the PR body to reflect that as the primary issue you'd like investigated. You can of course also open a new issue. |
I've created a new issue #41601 |
Preflight Checklist
Electron Version
25.4.0
What operating system are you using?
Other Linux
Operating System Version
NixOS
What arch are you using?
x64
Last Known Working Electron version
No response
Expected Behavior
When a scheme is set to stream: true, it should allow streaming of a resource. When a scheme is set to stream: false, it should disable streaming of a resource.
Actual Behavior
The opposite behavior is observed.
Testcase Gist URL
https://gist.github.com/jmercouris/fca7cba9d8e6b30a2f861268a3180cff
Additional Information
No response
The text was updated successfully, but these errors were encountered: