Skip to content

Storm Extended HTTP API (SYN-4803) #3366

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 106 commits into from
Oct 27, 2023
Merged

Storm Extended HTTP API (SYN-4803) #3366

merged 106 commits into from
Oct 27, 2023

Conversation

vEpiphyte
Copy link
Contributor

No description provided.

invisig0th and others added 22 commits September 27, 2023 14:18
@invisig0th
VERY WIP: thoughts on ext http api
be0bcbc
@vEpiphyte
add double init order test, minor tweaks
bd683c9
@vEpiphyte
WIP - start plumbing storm in, prims out
1a0f542
@vEpiphyte
WIP Steel thread
9db82bd
@vEpiphyte
Add a newlines thread
3712e95
@vEpiphyte
start path matching
18af8cf
@vEpiphyte
Update the reply() helper to be more intuitive for the simple use case
1756462
@vEpiphyte
Add .methods
21785f6
@vEpiphyte
Add schema
3318bef
@vEpiphyte
Serialize request information
872b964
@vEpiphyte
Add index support
52f7d22
@vEpiphyte
Remove some comments
95e8e53
@vEpiphyte
Add perms support
3ba5c9b
@vEpiphyte
Allow setting a api few values at creation time
5f44d34
@vEpiphyte
Clean up comments, add some discuss comments, rework the HttpExtApi s…
cfa7699 
…tate machine
@vEpiphyte
Add a few more todos
1f1a43a
@vEpiphyte
Add .view support to Http object
4904de7
@vEpiphyte
implement owner using heavy objects
c80bab5
@vEpiphyte
Fix CoreApi
a272a46
@vEpiphyte
Bunch of updates to address discussion items
9f4f2f3
@vEpiphyte
small updates
faa33e8
@vEpiphyte
Add a fsm test stub
cd38c90
@vEpiphyte vEpiphyte added enhancement reqChangelog requires changelog labels Oct 5, 2023
@vEpiphyte vEpiphyte mentioned this pull request Oct 5, 2023
Closed
Cisphyx
Cisphyx reviewed Oct 23, 2023
View reviewed changes
@vEpiphyte vEpiphyte requested a review from Cisphyx October 25, 2023 14:01
Cisphyx
Cisphyx reviewed Oct 25, 2023
View reviewed changes
@vEpiphyte vEpiphyte requested a review from Cisphyx October 25, 2023 21:35
Cisphyx
Cisphyx previously approved these changes Oct 25, 2023
View reviewed changes
@vEpiphyte vEpiphyte added this to the v2.15x.x milestone Oct 26, 2023
MichaelSquires
MichaelSquires reviewed Oct 26, 2023
View reviewed changes
Copy link
Contributor

@MichaelSquires MichaelSquires left a comment

Choose a reason for hiding this comment

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

Looks great! I think all my comments are just questions.

@vEpiphyte @MichaelSquires
Apply suggestions from code review
a7d228c 
Co-authored-by: blackout <blackout@vertex.link>
@vEpiphyte vEpiphyte modified the milestones: v2.15x.x, v2.153.0 Oct 27, 2023
@vEpiphyte vEpiphyte merged commit 12d7493 into master Oct 27, 2023
@vEpiphyte vEpiphyte deleted the http-ext-api-epiphyte branch October 27, 2023 17:12
@vEpiphyte vEpiphyte removed the reqChangelog requires changelog label Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MichaelSquires MichaelSquires MichaelSquires approved these changes

@invisig0th invisig0th Awaiting requested review from invisig0th

@Cisphyx Cisphyx Awaiting requested review from Cisphyx

Assignees
No one assigned
Labels
documentation enhancement
Projects
None yet
Milestone
v2.153.0
Development

Successfully merging this pull request may close these issues.
4 participants
@vEpiphyte @invisig0th @Cisphyx @MichaelSquires