Juliette comments

Results 2034 comments of


                                            Juliette

URL reformatting issue with Requests_IRI

**Action plan**: 1. Create a test case to proof or disproof that there is a bug. 2. If a failing test can be created: fix the bug. 3. Otherwise close...

[Task] Audit whether I18n domains are handled correctly everywhere

Also #759 (duplicate)

Missing Available Hook in Doc and Missing Hook Parameter

Before we actually action this, we should have a good look at the security implications of allowing access to the `$handle` (not just here).

Feature upload

@gdarko It's kind of hard to review code which is littered with unaddressed (automated) comments....

Create SECURITY.md

Hiya @JamieSlome and @ranjit-git, Thanks for bringing this to our attention. We'll get that sorted soonish (need to figure out an email address which can be used). In the mean...

Create SECURITY.md

For WordPress itself, HackerOne is definitely the preferred way. For Requests, we'll have a discussion about this in the near future, but until otherwise decided, HackerOne will work for the...

Create SECURITY.md

Just checking: did the issue ever get reported ? If so, would you mind posting a link so those of us who have access to the WP HackerOne account can...

Feature to set ip resolver type in request options

@ShivamYuvraj Thank you for contributing. Would it be possible to add some unit tests for this change ?

Optimize memory usage (potential memory eater)

Ideas: * Add a cache interface and separate the caching from the IRI class ? * Add a cache property/properties with appropriate cache setting/clearing options ? This should be looked...

Prevent cURL transport from leaking on Exception

Note for the next iteration of this PR: For testing the correct functioning of this change, the PHPUnit 9.3+ `Assert::assertIsClosedResource()` assertion can be used. This assertion is [polyfilled by the...