Skip to content

Allow expressions in HTTP query and headers #1080

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

Conversation

toxik
Copy link
Contributor

@toxik toxik commented Feb 10, 2025

Please specify parts of this PR update:

  • Specification
  • Schema
  • Examples
  • Extensions
  • Use Cases
  • Community
  • CTK
  • Other

Discussion or Issue link:
#1079

What this PR does:

Additional information:

@toxik toxik force-pushed the feat/http-expressions-in-headers-and-query branch from b6b4e37 to 7e4b9d7 Compare February 10, 2025 09:25
@cdavernas cdavernas changed the title feat(schema): allow expressions in HTTP query and headers Allow expressions in HTTP query and headers Feb 10, 2025
@cdavernas
Copy link
Member

cdavernas commented Feb 10, 2025

@toxik Thank you for your work!

Can I ask you to please open an issue describing the feature the PR addresses? It's important for book keeping purposes 😉

@cdavernas cdavernas added area: spec Changes in the Specification type: feature labels Feb 10, 2025
@cdavernas cdavernas added this to the v1.1.0 milestone Feb 10, 2025
@toxik
Copy link
Contributor Author

toxik commented Feb 10, 2025

@toxik Thank you for your work!

Can I ask you to please open an issue describing the feature the PR addresses? It's important for book keeping purposes 😉

Hey it's already opened at #1079

@cdavernas
Copy link
Member

cdavernas commented Feb 10, 2025

@toxik My bad, missed it by clicking directly on the PR email 😛

@cdavernas cdavernas linked an issue Feb 10, 2025 that may be closed by this pull request
Copy link
Member

@cdavernas cdavernas left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks ❤️

Copy link
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

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

Thanks! Can you add an example for this change so we can validate it in our CI? For reference: https://github.com/serverlessworkflow/specification/blob/main/examples/call-http-query-parameters.yaml

Issue serverlessworkflow#1079

Signed-off-by: Alexandru Georoceanu <alex@navigheaza.ro>
@toxik toxik force-pushed the feat/http-expressions-in-headers-and-query branch from 7e4b9d7 to 37fcaa4 Compare February 10, 2025 21:16
@toxik
Copy link
Contributor Author

toxik commented Feb 10, 2025

Thanks! Can you add an example for this change so we can validate it in our CI? For reference: https://github.com/serverlessworkflow/specification/blob/main/examples/call-http-query-parameters.yaml

Hey, sure thing – 37fcaa4

Copy link
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

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

Outstanding! Many thanks!

@ricardozanini ricardozanini merged commit 9689666 into serverlessworkflow:main Feb 10, 2025
3 checks passed
@toxik toxik deleted the feat/http-expressions-in-headers-and-query branch February 11, 2025 05:08
nyamathshaik pushed a commit to nyamathshaik/specification that referenced this pull request Mar 19, 2025
…workflow#1080)

Issue serverlessworkflow#1079

Signed-off-by: Alexandru Georoceanu <alex@navigheaza.ro>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>
ricardozanini added a commit that referenced this pull request Mar 19, 2025
* Fix #1076 - Make detail and title a runtime expression in the schema

Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Add string as anyOf

Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* feat(schema): allow expressions in HTTP query and headers (#1080)

Issue #1079

Signed-off-by: Alexandru Georoceanu <alex@navigheaza.ro>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multip… (#1082)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multiple YAML files

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(contributing): reverting pre-release versioning format

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

---------

Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Fix the description of flow directives and addressed few issues in dsl and dsl-reference (#1083)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multip… (#1082)

* update DSL version from alpha1, alpha2, alpha5 to 1.0.0 across multiple YAML files

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(contributing): reverting pre-release versioning format

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

---------

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl): fix a few issues in dsl and dsl-reference

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* update .gitignore to include .DS_Store

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl-reference): correct table formatting

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl): fix a few issues in dsl and dsl-reference

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* update .gitignore to include .DS_Store

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl-reference): correct table formatting

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl-reference): clarify flow directive descriptions

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* fix(dsl-reference): improve table formatting

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

* remove again .DS_Store

Signed-off-by: Gilles Barbier <gilles@infinitic.io>

---------

Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Fix broken link in README.md (#1084)

Signed-off-by: Michal Broz <mbroz@us.ibm.com>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Set task to support direct runtime expression

Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* updated workflow schema along with table fix

Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* updating the schema

Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* addressing PR comment reviews

Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* removing unwanted set example

Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Update dsl-reference.md

Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Update dsl-reference.md

Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Update dsl-reference.md

Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* reverting back original set example change

Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Update dsl-reference.md

Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* passing the set example as object

Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* reverting to original

Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Add a new file that compares the Serverless Workflow to other popular DSLs

Signed-off-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

* Fix a the `catalog` link in the `use` section table (#1088)

Signed-off-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>

---------

Signed-off-by: Ricardo Zanini <ricardozanini@gmail.com>
Signed-off-by: Nyamath Shaik <nyamath.shaik@uipath.com>
Signed-off-by: Alexandru Georoceanu <alex@navigheaza.ro>
Signed-off-by: Gilles Barbier <gilles@infinitic.io>
Signed-off-by: Michal Broz <mbroz@us.ibm.com>
Signed-off-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Co-authored-by: Ricardo Zanini <ricardozanini@gmail.com>
Co-authored-by: Alexandru Georoceanu <alex@navigheaza.ro>
Co-authored-by: Gilles Barbier <gilles@infinitic.io>
Co-authored-by: mbroz2 <mbroz@us.ibm.com>
Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: spec Changes in the Specification type: feature
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

HTTP Task should allow expressions in Headers and Query
3 participants