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

use curl_setop_array instead of curl_setop to set options #28

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Ibrahimzongo
Copy link

Hello,
Je viens d'ajouter de la doc à la fonction request dans XMLHttp, et une réorganisation du code avec curl_setopt_array que je trouve plus pratique.

@faso-dev
Copy link
Owner

faso-dev commented Jan 5, 2023

Salut Zongo et merci pour ma contribution.

Peut tu renommer ton commit en: use curl_setopt_array instead of curl_setopt to set options.

@Ibrahimzongo
Copy link
Author

Ibrahimzongo commented Jan 5, 2023 via email

@Ibrahimzongo Ibrahimzongo changed the title XMLHttp refactering use curl_setop_array instead of curl_setop to set options Jan 5, 2023
@Ibrahimzongo
Copy link
Author

C'est ok.

CURLOPT_POST => true,
CURLOPT_HTTPHEADER => $headers,
CURLOPT_RETURNTRANSFER => true,
CURLOPT_POSTFIELDS => $body,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tu peux supprimer l'espace blanc ici ?

@faso-dev
Copy link
Owner

faso-dev commented Jan 5, 2023

merci bien pour votre contribution

@Ibrahimzongo
Copy link
Author

Ibrahimzongo commented Jan 5, 2023 via email

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

Successfully merging this pull request may close these issues.

2 participants