knative / eventing

Event-driven application platform for Kubernetes
https://knative.dev/docs/eventing
Apache License 2.0
1.39k stars 582 forks source link

InMemoryChannel ingress: Reject unauthorized requests #7981

Open creydr opened 3 weeks ago

creydr commented 3 weeks ago

We need to verify in the channel receiver, that an request is authorized. Therefor we should do the following in the event receiver handler:

We should also add an e2e test for the above scenarios

Prerequisites:

Additional context:

Additional hints for new contributors before starting with this issue:

  1. When the issue has the Draft status, the issue is subject to change and thus should not be started to be worked on
  2. Make sure you've read and understood the CONTRIBUTING.md guidelines
  3. Make sure you're able to run Knative Eventing locally and run at least the unit tests.
  4. Feel free to raise any questions you have either directly here in the issue, in the #knative-eventing Slack channel or join the Eventing Workgroup Meeting
  5. When you feel comfortable with this issue, feel free to assign it to you (e.g. by commenting /assign). Please be aware that we might unassign you, if we don't see any progress from your side to give other contributors also a chance to work on this issue.
pratikkumar-mohite commented 3 weeks ago

I would like to work on it, @creydr could you please assign it to me?

7h3-3mp7y-m4n commented 2 weeks ago

/assign