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

Update apigee_edge_apiproduct_rbac.module bug fix issue 1098 #1101

Open
wants to merge 1 commit into
base: 3.x
Choose a base branch
from

Conversation

urbanenomad
Copy link

Changed from neutral to forbidden so that API Products do not show up on the assign operation or Create App when using the RBAC service to control access to API Products. This is to solve issue #1098

Changed from neutral to forbidden so that API Products do not show up on the assign operation or Create App when using the RBAC service to control access to API Products.  This is to solve issue apigee#1098
@kedarkhaire
Copy link
Collaborator

Hi @urbanenomad Thanks for creating the PR, we will check the implementation, but also it is mandatory that all tests pass in your repository, to make sure that the other working functionalities are unharmed.

Thanks!

@urbanenomad
Copy link
Author

I am a bit confused on some of these errors. There seems to be some kind of JWT token error in PHP 81./Drupal 10.3.x/X

"1) Drupal\Tests\apigee_edge_teams\Functional\ApigeeX\AccessTest::testAccess
Drupal\Core\Entity\EntityStorageException: "error":"invalid_grant","error_description":"Invalid JWT Signature."".

I am not sure why this error is occuring. I don't think I did any changes to the JWT signature? Do I need to test the code against a working apigee-x instance?

@kedarkhaire
Copy link
Collaborator

ror is occuring. I don't think I did any changes

Hi @urbanenomad
Yes the issue is not related to the changes you made, we just need to update the credentials for test. I will do that tomorrow with the working ones, and that issue will be solved.

I saw other issue on my test fork. Can you have a look at this link.

@kedarkhaire kedarkhaire self-assigned this Dec 17, 2024
@urbanenomad
Copy link
Author

urbanenomad commented Dec 17, 2024

I don't have an active apigee edge instance that seems to be something I need to test these unit tests. Is there a public one I can use to validate the unit tests from my fork?

@urbanenomad
Copy link
Author

urbanenomad commented Dec 18, 2024

@kedarkhaire another thing I see is that your kedarkhaire fork there seems to be 2 errors for similar issues for both AccessTest::testAcccess as well as UiTest::testUi. But in the original repo there only seems to be one error for the UiTest::testUi. Both seems to be based on the same error where it looks like the team_name doesn't seem to exist which is causing the undefined array key error? Also my apigee-x instance I have for testing the drupal portal seems to have monetization turned on so apigee-teams module does not seem to work in my portal since it is not designed to work with monetization turned on. Is there another apigee instance edge or x that we can use to test this?

@kedarkhaire
Copy link
Collaborator

Hi @urbanenomad
Thanks for your inputs, I will check and update.

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