Skip to content

Conversation

@zkat
Copy link
Contributor

@zkat zkat commented Oct 21, 2025

No description provided.

@vcarvajal-sigsci
Copy link
Collaborator

🥳

@zkat zkat force-pushed the zkat/inspect branch 9 times, most recently from c59a0e4 to 7e8b441 Compare December 12, 2025 04:33
@zkat zkat force-pushed the zkat/inspect branch 8 times, most recently from e8a2818 to 3a41d8b Compare December 12, 2025 21:26
@zkat zkat requested a review from TartanLlama December 12, 2025 21:26
@zkat zkat marked this pull request as ready for review December 12, 2025 21:27
@zkat zkat enabled auto-merge (squash) December 12, 2025 21:29
@zkat zkat merged commit 98f45e4 into main Dec 15, 2025
26 of 29 checks passed
@zkat zkat deleted the zkat/inspect branch December 15, 2025 17:08
/**
* Inspect a Request using the [Fastly Next-Gen WAF](https://docs.fastly.com/en/ngwaf/).
*/
export function inspect(request: Request): InspectResponse;
Copy link

Choose a reason for hiding this comment

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

this is missing the config argument

Copy link
Contributor Author

Choose a reason for hiding this comment

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

#1274 oops. Thanks for the heads-up!

zkat added a commit that referenced this pull request Jan 7, 2026
@zkat zkat mentioned this pull request Jan 7, 2026
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.

5 participants