#57029 closed defect (bug) (invalid)
Insecure links needs to be replaced with the secure ones.
| Reported by: | krunal265 | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | External Libraries | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | docs |
Description
I came across the file where the insecure link was added even if the secure link is active on the web.
Attachments (1)
Change History (7)
#3
@
4 years ago
- Component General → External Libraries
- Milestone Awaiting Review
- Resolution → invalid
- Status new → closed
SimplePie is an external library, and this should be reported upstream directly to them:
https://github.com/simplepie/simplepie
I'm marking this as invalid, noting that if it's changed upstream in the project it'll be merged to WordPress in it's next release.
#4
@
4 years ago
Also, When submitting patches, please do a more complete review rather than selecting a singular instance.
There's numerous other http:// strings in the SimplePie codebase, including one on the very next line after 57029.patch.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
patch added.