![[personal profile]](https://www.dreamwidth.org/img/silk/identity/user.png)
Branch: refs/heads/develop
Home: https://github.com/dreamwidth/dw-free
Commit: 5d34b9321e2764c6e974cf3edf19e6e2153b466b
https://github.com/dreamwidth/dw-free/commit/5d34b9321e2764c6e974cf3edf19e6e2153b466b
Author: Afuna <fu@dreamwidth.org>
Date: 2015-02-20 (Fri, 20 Feb 2015)
Changed paths:
M cgi-bin/DW/Controller/Journal/EmbeddedContent.pm
M cgi-bin/DW/Hooks/EmbedWhitelist.pm
M cgi-bin/LJ/CleanHTML.pm
M cgi-bin/LJ/EmbedModule.pm
Log Message:
-----------
[#1213] Rewrites http:// video embeds to use protocol relative URLs
* ... only if they support them (all the big sites do, smaller ones may
or may not). We don't want to force to https if it isn't supported
because the behavior is unreliable.
An http embed on https may throw up a browser warning.
A non-working https embed may, e.g., show a cryptic error message
about internal server error, a blank page, or (in one case) an
unrelated internal service
* added arguments so that we only change the URL if we're going to
display the embed (as opposed to editing the embed, or cleaning for
saving in the database).
Fixes #1213.
Commit: 5093e6a8b2baa2e1cfc0c129324c9ced9ea0d254
https://github.com/dreamwidth/dw-free/commit/5093e6a8b2baa2e1cfc0c129324c9ced9ea0d254
Author: Mark Smith <mark@qq.is>
Date: 2015-02-20 (Fri, 20 Feb 2015)
Changed paths:
M cgi-bin/DW/Controller/Journal/EmbeddedContent.pm
M cgi-bin/DW/Hooks/EmbedWhitelist.pm
M cgi-bin/LJ/CleanHTML.pm
M cgi-bin/LJ/EmbedModule.pm
Log Message:
-----------
Merge pull request #1240 from afuna/embeds-https-bug-1213
[#1213] Rewrites http:// video embeds to use protocol relative URLs
Compare: https://github.com/dreamwidth/dw-free/compare/bc76e0cf9442...5093e6a8b2ba
Home: https://github.com/dreamwidth/dw-free
Commit: 5d34b9321e2764c6e974cf3edf19e6e2153b466b
https://github.com/dreamwidth/dw-free/commit/5d34b9321e2764c6e974cf3edf19e6e2153b466b
Author: Afuna <fu@dreamwidth.org>
Date: 2015-02-20 (Fri, 20 Feb 2015)
Changed paths:
M cgi-bin/DW/Controller/Journal/EmbeddedContent.pm
M cgi-bin/DW/Hooks/EmbedWhitelist.pm
M cgi-bin/LJ/CleanHTML.pm
M cgi-bin/LJ/EmbedModule.pm
Log Message:
-----------
[#1213] Rewrites http:// video embeds to use protocol relative URLs
* ... only if they support them (all the big sites do, smaller ones may
or may not). We don't want to force to https if it isn't supported
because the behavior is unreliable.
An http embed on https may throw up a browser warning.
A non-working https embed may, e.g., show a cryptic error message
about internal server error, a blank page, or (in one case) an
unrelated internal service
* added arguments so that we only change the URL if we're going to
display the embed (as opposed to editing the embed, or cleaning for
saving in the database).
Fixes #1213.
Commit: 5093e6a8b2baa2e1cfc0c129324c9ced9ea0d254
https://github.com/dreamwidth/dw-free/commit/5093e6a8b2baa2e1cfc0c129324c9ced9ea0d254
Author: Mark Smith <mark@qq.is>
Date: 2015-02-20 (Fri, 20 Feb 2015)
Changed paths:
M cgi-bin/DW/Controller/Journal/EmbeddedContent.pm
M cgi-bin/DW/Hooks/EmbedWhitelist.pm
M cgi-bin/LJ/CleanHTML.pm
M cgi-bin/LJ/EmbedModule.pm
Log Message:
-----------
Merge pull request #1240 from afuna/embeds-https-bug-1213
[#1213] Rewrites http:// video embeds to use protocol relative URLs
Compare: https://github.com/dreamwidth/dw-free/compare/bc76e0cf9442...5093e6a8b2ba