cloudflare / cloudflare/cloudflare-php
Changing guzzle adapter options
- Vorherrschende Sprache
- PHP
- Sterne
- 674
- Forks
- 268
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
I've come across this issue while testing your api. Guzzle throws an exception complaining there's no ca file on a windows box. For those OS' without a default known ca file, [see guzzle verify](https://guzzle.readthedocs.io/en/stable/request-options.html#verify), it is not possible to set a default ca-file without altering the php.ini or PATH environment. To solve this issue,
**Either**
//to set options to the guzzle client
public getClient() { return $this->client; }
**Or**
public function __construct(Auth $auth, string $baseURI = null, $guzzleConfig = [])
{
if ($baseURI === null) {
$baseURI = 'https://api.cloudflare.com/client/v4/';
}
$headers = $auth->getHeaders();
$config = [
'base_uri' => $baseURI,
'headers' => $headers,
'Accept' => 'application/json'
] + $guzzleConfig;
$this->client = new Client($config);
}
**Or**
public function __construct(Auth $auth, string $baseURI = null, $caFile = false)
{
if ($baseURI === null) {
$baseURI = 'https://api.cloudflare.com/client/v4/';
}
$headers = $auth->getHeaders();
$config = [
'base_uri' => $baseURI,
'headers' => $headers,
'Accept' => 'application/json'
];
if ($caFile !== false) $config['verify'] = $caFile;
$this->client = new Client($config);
}
can be done.
Beitragsleitfaden
Rechercherichtung
Lokalisieren Sie den Konstruktor des API-Clients und die im Issue gezeigte Erstellung des Guzzle-Clients. Prüfen Sie, wie seine Basis-URI und Header derzeit konfiguriert werden, und bestimmen Sie anschließend, welche Form der vorgeschlagenen Option zur bestehenden öffentlichen API passt. Die Aufgabe ist erledigt, wenn Aufrufer die CA-Datei oder andere Guzzle-Optionen konfigurieren können, ohne php.ini oder Umgebungsvariablen zu bearbeiten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- api
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100