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

Request body deserialization fix in IAdmissionWebhook #533

Merged
merged 2 commits into from Feb 22, 2023
Merged

Request body deserialization fix in IAdmissionWebhook #533

merged 2 commits into from Feb 22, 2023

Conversation

tomasfabian
Copy link
Contributor

@tomasfabian tomasfabian commented Feb 17, 2023

fixes #531

@tomasfabian
Copy link
Contributor Author

Hi Christoph @buehler,
could you please have a look on this PR, please? We had to migrate to v7.x.x in order to get this fix, but we stumbled on this new issue which prevents us from fixing our operator.

Thank you very much!

@buehler buehler merged commit be27d6c into buehler:master Feb 22, 2023
@tomasfabian
Copy link
Contributor Author

tomasfabian commented Feb 22, 2023

Thank you @buehler for merging this PR! Could you please check the semantic release, please? It failed.

@buehler
Copy link
Owner

buehler commented Feb 22, 2023

Yeah, I saw that. Already creating a new version for it...

@buehler
Copy link
Owner

buehler commented Feb 23, 2023

Finally. Version 7.0.3 of the dotnet sdk (7.0.200) changed a flag while creating packages. This lead to errors during build.

@tomasfabian
Copy link
Contributor Author

Finally. Version 7.0.3 of the dotnet sdk (7.0.200) changed a flag while creating packages. This lead to errors during build.

Thank you very much for you effort Christoph!

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.

[bug]: Mutating WebHook is throwing a Json exception
3 participants