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

Jump to bottom

Misleading redirect shown for HTTP API request #738

Open
johnbillion opened this issue Mar 1, 2023 · 4 comments · May be fixed by #790
Open

Misleading redirect shown for HTTP API request #738

johnbillion opened this issue Mar 1, 2023 · 4 comments · May be fixed by #790
Labels
bug collector:http

Comments

@johnbillion
Copy link
Owner

An HTTP API request to the Gravity Forms API with a trailing question mark gets marked as having been redirected on the receiving end because the trailing question mark is stripped somewhere. Need to investigate why this is flagged as a redirect.
@crstauf
Copy link
Contributor

crstauf commented Jun 21, 2023

It would technically be a redirect, would it not?

@crstauf
Copy link
Contributor

crstauf commented Jul 5, 2023

Image

@crstauf crstauf linked a pull request Jul 10, 2023 that will close this issue
Open
@crstauf
Copy link
Contributor

crstauf commented Jul 10, 2023 •
edited

@johnbillion Fixed identifying as a redirect in #790. Would you also want the ? removed from the URL if the query string is empty?

2023-07-10T222821 
add_action( 'init', static function () {
	wp_remote_get( 'https://querymonitor.com/?' );
} );

@crstauf
Copy link
Contributor

crstauf commented Jul 11, 2023 •
edited

Would you also want the ? removed from the URL if the query string is empty?

Went ahead and removed it in 3cfdeec.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug collector:http
Projects
Query Monitor
Status: Bug
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes misleading redirect shown for HTTP API request crstauf/query-monitor
2 participants
@johnbillion @crstauf