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

Possibility to run cache warm up also with activated basic http auth #783

Open
aboritskiy opened this issue Sep 29, 2020 · 4 comments
Open

Comments

@aboritskiy
Copy link

Is your feature request related to a problem? Please describe.
If production or staging env is behind basic http auth - cache warm up will fail cause it will not get access to the server.
This makes it problematic to test if cache warm up actually works which preparing the website for go live without granting access to general public.

Describe the solution you'd like
One can pass auth configuration to Guzzle Http lib, this way Guzzle will be able to authenticate against password protected magento pages and warm up cache.

Describe alternatives you've considered
Possible alternative is simply always whitelist request coming from the host itself.

Additional context
No additional context except for the fact that Guzzle Http actually supports that feature, one just needs to pass proper config in it.

@m2-community-project m2-community-project bot added this to Ready for Grooming in Backlog Sep 29, 2020
@aboritskiy
Copy link
Author

Possible alternative is simply always whitelist request coming from the host itself.

just tried that - it is not easy for the env that is behind Fastly, what will be the real container IP when it will hit the host? Is that container IP static? All those are shaky assumptions and better way is to allow passing basic http auth through.

@andriyShevtsov
Copy link
Contributor

@aboritskiy is this request still up to date?

@aboritskiy
Copy link
Author

@andriyShevtsov I think it is still up to date, as far as I know, cache warm up still breaks if website is behind basic auth.

@BaDos
Copy link
Contributor

BaDos commented Sep 6, 2022

The internal task MCLOUD-9281 was created

@m2-community-project m2-community-project bot moved this from Ready for Grooming to Ready for Development in Backlog Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Backlog
Ready for Development
Development

No branches or pull requests

5 participants