feat: As a user, I want to include the request body in the opa-input, so that I can reason about its contents

[wistefan]

Description

In order to make decisions based on the request body, the OPA-Plugin will also forward the body when configured to do so.

Fixes #11387

Checklist

• [x] I have explained the need for this PR and the problem it solves
• [x] I have explained the changes or the new features added to this PR
• [x] I have added tests corresponding to this change
• [x] I have updated the documentation to reflect this change
• [x] I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

[wistefan]

Is there anything I can do to get this PR forward?

[github-actions[bot]]

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions.

[moonming]

@wistefan Please synchronize the master code to trigger all CI

[wistefan]

@wistefan Please synchronize the master code to trigger all CI

@moonming Thank you for the review, I updated.

[LuciaCabanillasRodriguez]

Hello! If I want this new implementation with the request body, what should I do?

[Baoyuantop]

Hi @wistefan, please synchronize the latest master branch code to trigger the test.

[Baoyuantop]

Hi @wistefan, please fix failed ci

[Baoyuantop]

Hi @wistefan, do you have time to continue working on this PR?

[wistefan]

@Baoyuantop Yes, most likely beginning of next week

[wistefan]

@Baoyuantop Could you please appove the ci workflows?

[Baoyuantop]

@wistefan, please fix failed ci

[LuciaCabanillasRodriguez]

Hello! What is the status of this?

[Baoyuantop]

Need to fix the failed CI, waiting for a response from the author.

[LuciaCabanillasRodriguez]

Hello! Any news?

[wistefan]

Hi, sorry I will take care as soon as I find time. @LuciaCabanillasRodriguez help would be very welcome, do you have the time to take a look at the test failures?

[LuciaCabanillasRodriguez]

Hi, sorry I will take care as soon as I find time. @LuciaCabanillasRodriguez help would be very welcome, do you have the time to take a look at the test failures?

Hello! I tried to push a file to fix one of the issues, but I received a forbidden message. In any case, I solved it locally by running:

make lint ./utils/reindex t/plugin/opa3.t

[LuciaCabanillasRodriguez]

Regarding the error in t/discovery/consul_dump.t, I see that the test fails because the expected response from Consul isn’t being returned: got: '' expected: '{"service_a":[{"host":"127.0.0.1","port":30511,"weight":1}]}'

It seems like either the service_a isn’t being registered properly in Consul during the test, or the endpoint isn’t responding as expected.

As for the other warnings and errors like: failed to do SSL handshake: certificate verify failed http_init(): failed to load the configuration: connection refused

These appear to be related to missing services (like Consul or Etcd) or SSL verification issues, but I’m not entirely sure how to fix it with certainty.

[LuciaCabanillasRodriguez]

Good! I could remove one issue, the other one remains! Any help?? I would need some guidance on how to fix it

[Baoyuantop]

Maybe we can merge the master branch.

[LuciaCabanillasRodriguez]

Maybe we can merge the master branch. Is there any update here?

[github-actions[bot]]

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions.