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.

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

Proxy system setting ignored in MODX3 #16538

Closed
MaticSulc opened this issue Mar 12, 2024 · 0 comments · Fixed by #16539
Closed

Proxy system setting ignored in MODX3 #16538

MaticSulc opened this issue Mar 12, 2024 · 0 comments · Fixed by #16539
Labels
bug The issue in the code or project, which should be addressed.

Comments

@MaticSulc
Copy link
Contributor

MaticSulc commented Mar 12, 2024

Bug report

Summary

While using the proxy_host and adjacent system settings, them seem to get ignored in MODX 3. There's already a community thread active on the topic here.

Step to reproduce

Set the proxy_host, port and other system settings and open the Extras page (or do a curl request)

Observed behavior

Proxy settings are ignored, and it goes in timeout due to un-reachability.

Expected behavior

It should respect proxy settings and use them.

Environment

MODX 3.0.4-pl, PHP 8.1.

@MaticSulc MaticSulc added the bug The issue in the code or project, which should be addressed. label Mar 12, 2024
opengeek pushed a commit that referenced this issue May 20, 2024
### What does it do?
It uses the MODX proxy system settings for the PSR-7 service.

### Why is it needed?
When behind a proxy, this allows search/listing of Extras.

### How to test
Set all required proxy settings, open the Extras manager. If access to
the Internet is only available via a proxy, it should work as expected.

### Related issue(s)/PR(s)
Resolves #16538

---------

Co-authored-by: Matic Šulc <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue in the code or project, which should be addressed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant