Skip to content
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.

Sign up for GitHub

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

Add a warning when Cloudflare blocks REST API requests #8640

Merged
merged 14 commits into from
Aug 8, 2018
Merged

Add a warning when Cloudflare blocks REST API requests #8640

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
1bd7f2a
Catch a Cloudflare response, and redirect to the Classic editor..
pento Aug 7, 2018
6148a05
Ensure the url package is loaded when api-fetch needs it.
pento Aug 7, 2018
b8af9ea
Show a message in the Classic Editor when we redirect there from a Cl…
pento Aug 7, 2018
598a73a
Don't auto-redirect to Classic, give a link, instead.
pento Aug 7, 2018
d41f658
Linting.
pento Aug 7, 2018
8c384cc
Pinking shears
pento Aug 7, 2018
4ad8095
Fix the incorrect Cloudflare rule ID being listed.
pento Aug 7, 2018
671208e
Mock Resonse.clone() in the tests.
pento Aug 7, 2018
9191016
Make checkCloudflareError() more resilient to non-strings being passe…
pento Aug 7, 2018
2219590
Use typeof instead of checking for .indexOf on the error to check tha…
pento Aug 8, 2018
fbe7d6f
s/login/log in/
pento Aug 8, 2018
483beab
Improve translator comments.
pento Aug 8, 2018
9c70019
Improve readability of the JSON parsing error handling
pento Aug 8, 2018
468d024
Add a comment explaining why the response is being cloned.
pento Aug 8, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/api-fetch/src/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ function registerMiddleware( middleware ) {
}

function checkCloudflareError( error ) {
if ( error.indexOf( 'Cloudflare Ray ID' ) >= 0 ) {
if ( error.indexOf && error.indexOf( 'Cloudflare Ray ID' ) >= 0 ) {
Copy link
Member

@gziolo gziolo Aug 7, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or maybe:

 typeof error === 'string'

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

throw {
code: 'cloudflare_error',
};
Expand Down