-
-
Notifications
You must be signed in to change notification settings - Fork 923
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
feat: always use https for loremflickr #1034
feat: always use https for loremflickr #1034
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1034 +/- ##
=======================================
Coverage 99.67% 99.67%
=======================================
Files 2119 2119
Lines 227439 227432 -7
Branches 982 982
=======================================
- Hits 226691 226687 -4
+ Misses 728 725 -3
Partials 20 20
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we not consider this a breaking change?
IMO it's okay, only very few customers would use this last parameter and even if they would use it, they only get a quick compile error in TS if they even have TS. In JS the last parameter would just be ignored. |
When using https for you testpage you can only use the imageUrl function, all the other image helper functions will generate http:// urls.
This results in the image being blocked by the browser:
https image urls also work when the the page is hosted on http, in fact the http image urls will be redirected to https by lorumflicker.
This PR changes the default to
https://
and because the image is served over https anyway i've removed thehttps?: boolean
parameter instead of swapping the default value.The only code change is in the
imageUrl
function( line 93), the other changes are inside comments.