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

Add support for foreign function callbacks. #290

Merged
merged 5 commits into from
Jan 15, 2025

Conversation

casimiro
Copy link
Contributor

No description provided.

@casimiro casimiro force-pushed the foreign-function-callback branch 2 times, most recently from a7cb38f to 501e78c Compare November 18, 2024 21:28
src/types.rs Outdated Show resolved Hide resolved
src/dispatcher.rs Outdated Show resolved Hide resolved
src/traits.rs Outdated Show resolved Hide resolved
src/dispatcher.rs Outdated Show resolved Hide resolved
@casimiro casimiro force-pushed the foreign-function-callback branch from 501e78c to 086d863 Compare November 26, 2024 11:28
@casimiro
Copy link
Contributor Author

Thank you for the review @PiotrSikora! I've addressed the comments.

Copy link
Member

@PiotrSikora PiotrSikora left a comment

Choose a reason for hiding this comment

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

Thanks!

@casimiro casimiro force-pushed the foreign-function-callback branch from 086d863 to d620329 Compare December 2, 2024 14:04
@PiotrSikora PiotrSikora changed the title Add support for foreign function callback. Add support for foreign function callbacks. Jan 15, 2025
@PiotrSikora PiotrSikora merged commit ae38c0f into proxy-wasm:main Jan 15, 2025
23 checks passed
thibaultcha pushed a commit to Kong/proxy-wasm-rust-sdk that referenced this pull request Jan 23, 2025
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