Problem/Motivation
Replace http urls with https urls of the respective sites in core
Steps to reproduce
Proposed resolution
Replace http urls with https urls of the respective sites in core
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Issue fork drupal-3399840
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
bhanu951 commentedComment #4
bhanu951 commentedComment #5
cilefen commentedWe don't modify vendored code.
Comment #7
bhanu951 commented@saurabh_jain I don't get why you reverted those changes?
This issue is about making exact those changes.
Replacing all http URLs with https in core.
Comment #8
mikelutzYeah, I don't think #5 was about our code having links to third party websites. It was about modifications in assets/vendor, which is vendor code that we directly copy into our repo. Those files should not diverge from the vendor versions.
Comment #9
bhanu951 commentedShould this be handled as a single issue or should it be split into multiple smaller issues ?
IF smaller issues on what criteria they should be split ?
Based on Domain names ? or module components or any other ?
I would suggest let's make it a strictly first time contributor issues to get new contributors familiar with contribution.
Comment #10
quietone commentedYes, let's re-scope this into small, easier to review issues. Ideally, the scope is by the type of fix which makes it much easier for reviewers. I haven't looked closed enough but it may be possible to have an issue for user facing strings, one for comments, one for tests, and one for production code. I did see there are a lot of changes to .svg files, which could be a separate issue. So, this issue could be the Meta with children. And when, the children are fixed this could be used to fix any stragglers that have crept in to the code base.
Some other thoughts
1. Any ideas on how to prevent 'http:' in the future
2. Are there tests that are specifically testing http that should not be changed?
3. the migrate test fixtures drupal6.php and durpal7.php are not to be changed.
Comment #11
mikelutzFor lack of my 7am brain being able to come up with a clever way to do this in bash, running
on the original MR gives
along with ~80 additional domains with < 5 usages
The numbers are a bit off from the actual work between codebase changes and the vendor changes to be undone. Also the xmnls in the svg files should not be changed, https://www.w3.org/2000/svg is an identifier, not a url, and it should not be https. Still probably a good baseline to start with. Having discussed scoping child issues of metas like this several times with @xjm in the recent past, I'm fairly confident we should be doing this by domain for the top domains and then catch the one-offs in a separate issues or issues as it makes sense. These are probably not novice issues (at least not all) as there are many places where this should not be changed at all. Let's start with a child issues for
and go from there. We can also probably standardize on replacing http://www.example.com with https://example.com and http://drupal.org with https://www.drupal.org while we are at it.
As far as preventing reocurrances, I think we will have to turn to phpstan or codesniffer, and I'm not 100% sure which is better, but I would expect it is codesniffer.
Comment #12
cilefen commentedComment #13
quietone commented@mikelutz, thanks for the script and the numbers. Doing this by URL makes sense to me now.
Comment #14
quietone commented