Skip to content

SEC: fix exploitable template-injection surface (2/n)#370

Merged
Cadair merged 1 commit intoOpenAstronomy:mainfrom
neutrinoceros:sec/no-template-injection-2
Mar 9, 2026
Merged

SEC: fix exploitable template-injection surface (2/n)#370
Cadair merged 1 commit intoOpenAstronomy:mainfrom
neutrinoceros:sec/no-template-injection-2

Conversation

@neutrinoceros
Copy link
Contributor

extracted from #368
ref #364

@neutrinoceros neutrinoceros force-pushed the sec/no-template-injection-2 branch 2 times, most recently from bc2577b to 9e19594 Compare March 9, 2026 12:16
@neutrinoceros neutrinoceros marked this pull request as ready for review March 9, 2026 12:19
@Cadair
Copy link
Member

Cadair commented Mar 9, 2026

This now needs a rebase, probably easier if we keep this to one bigger PR?

@neutrinoceros
Copy link
Contributor Author

not sure. I'd prefer to have all the known-working bits merged as soon as possible if you don't mind, since I cannot iterate on workflows locally.

@neutrinoceros neutrinoceros force-pushed the sec/no-template-injection-2 branch from 9e19594 to 3f1e9ea Compare March 9, 2026 13:44
@neutrinoceros
Copy link
Contributor Author

(also in this instance, the merge conflict had nothing to do with my other PRs, which makes sense since they don't touch the same lines)

@Cadair Cadair merged commit 45e1c19 into OpenAstronomy:main Mar 9, 2026
142 of 144 checks passed
@neutrinoceros neutrinoceros deleted the sec/no-template-injection-2 branch March 9, 2026 13:54
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