Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

H1rono
Copy link
Contributor

@H1rono H1rono commented Jul 10, 2024

#208 (comment)
depends on #227

@H1rono
Copy link
Contributor Author

H1rono commented Jul 15, 2024

これstack overflowで落ちてるんですがどうしましょうか...
RUST_MIN_STACKをでかくしたら通ることは確認できました

@sksat
Copy link
Owner

sksat commented Nov 1, 2024

ほんとだ test で stack overflow になりますね(スレ遅)

@H1rono
Copy link
Contributor Author

H1rono commented Dec 24, 2024

これ、実際にHTTP POSTリクエストを処理する場合はヘッダー X-GitHub-Event の値からペイロードの構造がわかるので、テストの想定するユースケースが不適切とも言えるんですよね

https://github.com/H1rono/bot-cnvtr/blob/616c4b663f3712cfb6129b141ee312b6393e1651/app/wh-handler/src/handler_impl/github.rs#L53-L66

CIでRUST_MIN_STACKをいじるのもいいですが、テストコードに手を入れるのもアリだと思います

(スレ遅 🙏)

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