Skip to content

Fixes #420 - Add http invocation function definition #647

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

Closed
wants to merge 14 commits into from

Conversation

ricardozanini
Copy link
Member

@ricardozanini ricardozanini commented Jun 27, 2022

Signed-off-by: Ricardo Zanini [email protected]

Many thanks for submitting your Pull Request ❤️!

Please specify parts of this PR update:

  • Specification
  • Schema
  • Examples
  • Extensions
  • Roadmap
  • Use Cases
  • Community
  • TCK
  • Other

Discussion or Issue link: #420

What this PR does / why we need it:

  • Limit the rest function to OpenAPI specification descriptors
  • Add the new http function type for simple use cases that requires an HTTP invocation. REST operations can still be performed if well defined in the function definition. Of course, simple use cases are also supported.

Special notes for reviewers:
This is a WIP.

Additional information: Based on https://www.rfc-editor.org/rfc/rfc9110.html

@ricardozanini ricardozanini linked an issue Jun 27, 2022 that may be closed by this pull request
@ricardozanini ricardozanini added change: feature New feature or request. Impacts in a minor version change area: spec Changes in the Specification schema labels Jun 27, 2022
@ricardozanini ricardozanini added this to the v0.9 milestone Jun 28, 2022
@ricardozanini
Copy link
Member Author

@cdavernas mind taking a look? I think we can try instantiating a few use cases to verify if this architecture is suitable IRL.

@cdavernas
Copy link
Member

@ricardozanini Will do tomorrow morning!

@tsurdilo
Copy link
Contributor

@ricardozanini Will do tomorrow morning!

also don't forget that this is spec and the dsl needs to be user friendly first ;)

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 change: feature New feature or request. Impacts in a minor version change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants