You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/06/09 17:27:52 UTC

[GitHub] [trafficserver] brbzull0 opened a new pull request, #8901: uri_signing plugin: Fix missing payload validation for the iss field.

brbzull0 opened a new pull request, #8901:
URL: https://github.com/apache/trafficserver/pull/8901

   Should fix the crash for https://github.com/apache/trafficserver/issues/8874


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [trafficserver] zwoop commented on pull request #8901: uri_signing plugin: Fix missing payload validation for the iss field.

Posted by GitBox <gi...@apache.org>.
zwoop commented on PR #8901:
URL: https://github.com/apache/trafficserver/pull/8901#issuecomment-1157015566

   Cherry-picked to v9.2.x


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [trafficserver] brbzull0 commented on pull request #8901: uri_signing plugin: Fix missing payload validation for the iss field.

Posted by GitBox <gi...@apache.org>.
brbzull0 commented on PR #8901:
URL: https://github.com/apache/trafficserver/pull/8901#issuecomment-1151415151

   @dsouza93 you may be interested in this.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [trafficserver] dsouza93 commented on pull request #8901: uri_signing plugin: Fix missing payload validation for the iss field.

Posted by GitBox <gi...@apache.org>.
dsouza93 commented on PR #8901:
URL: https://github.com/apache/trafficserver/pull/8901#issuecomment-1152601043

   I am taking a look at this today @brbzull0! Thanks a ton for posting this up.
   
   It may take me just a little more time. I have a bunch of fixes for this plugin that are in our internal repos that I though I pushed to the open source repo here. I want to just go through them and ensure that this is not one of those and that I didn't miss anything else.
   
   That being said, I'll definitely get around to testing this and approving by Wednesday next week!
   
   Also thank you so much for adding that unit test as well! 🙏 
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [trafficserver] traeak merged pull request #8901: uri_signing plugin: Fix missing payload validation for the iss field.

Posted by GitBox <gi...@apache.org>.
traeak merged PR #8901:
URL: https://github.com/apache/trafficserver/pull/8901


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org