Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Proxy Unterstützung für Automator checkForUpdates() #7681

Closed
BugBuster1701 opened this issue Mar 11, 2015 · 7 comments
Closed

Proxy Unterstützung für Automator checkForUpdates() #7681

BugBuster1701 opened this issue Mar 11, 2015 · 7 comments
Assignees
Labels
Milestone

Comments

@BugBuster1701
Copy link
Contributor

Ebenso wie die Repository Klassen wäre es wünschenswert wenn auch bei Automator checkForUpdates() der Hook genutzt werden würde.
Ich habe lange gesucht warum bei mir nie ein Update Hinweis kam. (Firmen Zwangsproxy)
Die Zeile 41 https://github.com/contao/core/blob/master/system/modules/core/library/Contao/Automator.php#L41

            $objRequest = new \Request();

ersetzen durch

        // HOOK: proxy module
        if (Config::get('useProxy')) {
            $objRequest = new \ProxyRequest();
        } else {
            $objRequest = new \Request();
        }

(entnommen aus der RepositoryManager Klasse)

@ciaobello
Copy link

+1

@leofeyer leofeyer added this to the 3.5.0 milestone Mar 20, 2015
@leofeyer leofeyer self-assigned this Mar 20, 2015
@leofeyer
Copy link
Member

Geändert in 94dd80f.

@Toflar
Copy link
Member

Toflar commented Mar 24, 2015

Can somebody explain me where the class ProxyRequest comes from? I've never heard from it and I can't find the class at all?!?!

@leofeyer
Copy link
Member

@Toflar
Copy link
Member

Toflar commented Mar 24, 2015

kk

@aschempp
Copy link
Member

Should we add a in_array('proxy', \ModuleLoader::getActiveModules()) too?

@leofeyer
Copy link
Member

Yes, maybe. I have also thought about adding the class to the core. Maybe @BugBuster1701 can prepare a pull request?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants