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

feat: add connect-timeout option per request #3306

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

Conversation

zikani03
Copy link

@zikani03 zikani03 commented Oct 14, 2024

Added a connect-timeout option to allow users to specify a connect-timeout for each request in a hurl file.

Closes #3163

@zikani03 zikani03 force-pushed the 3163-connect-timeout-per-request branch from 59036a9 to cbbf1ae Compare October 14, 2024 22:51
@jcamiel
Copy link
Collaborator

jcamiel commented Oct 15, 2024

Hi @zikani03 don't worry about the failed pipelines, we have some issues to fix on it, I will ping you when evrything comes back to normal 😔

@lepapareil
Copy link
Collaborator

/rebase

@hurl-bot
Copy link
Collaborator

🕗 /rebase is running, please wait for completion.

Added a connect-timeout option to allow users to specify a
connect-timeout for each request in a hurl file.

Closes Orange-OpenSource#3163
@hurl-bot
Copy link
Collaborator

🔨 Auto rebase from Orange-OpenSource/hurl/master succeeds, zikani03/hurl/3163-connect-timeout-per-request now embeds these commits:

  • 5a52728 Fix powershell install in checks ci
  • deeaf02 Fix python venv in checks ci
  • 12dc0cd Disable PAM for sudo in Fedora ci

@hurl-bot hurl-bot force-pushed the 3163-connect-timeout-per-request branch from cbbf1ae to a3e9b1a Compare October 15, 2024 19:38
@lepapareil
Copy link
Collaborator

lepapareil commented Oct 15, 2024

Hi @zikani03, all issues have been fixed on ci, you can rely on the pipelines once again 😀.

@jcamiel
Copy link
Collaborator

jcamiel commented Oct 16, 2024

Hi @zikani03 you need to do some modifications (the CI/CD has failed because some new branch code is not coverered in hurlfmt).

What you need to do:

  • modify integration/hurlfmt/tests_export/options.hurl and add a line with connect_timeout: 60s for instance
  • update the exported Hurl file in JSON: integration/hurlfmt/tests_export/options.json: run hurlfmt --out json integration/hurlfmt/tests_export/options.hurl
  • update the exported Hurl file in HTML: integration/hurlfmt/tests_export/options.html: run hurlfmt --out html integration/hurlfmt/tests_export/options.hurl

The CI/CD should be OK after that,

Thanks !

@zikani03
Copy link
Author

Hi @jcamiel , thanks for the direction - I have updated the hurlfmt test files, also added the option to the hurl.grammar

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.

Configure --connect-timeout per request
4 participants