magento / magento2

Prior to making any Submission(s), you must sign an Adobe Contributor License Agreement, available here at: https://opensource.adobe.com/cla.html. All Submissions you make to Adobe Inc. and its affiliates, assigns and subsidiaries (collectively “Adobe”) are subject to the terms of the Adobe Contributor License Agreement.
http://www.magento.com
Open Software License 3.0
11.55k stars 9.32k forks source link

Unable to do a PUT call for Matrix Sub items using OAuth Connection . #38769

Closed saikalyan425 closed 4 months ago

saikalyan425 commented 5 months ago

Preconditions and environment

Steps to reproduce

Create a Matrix parent item with one sub matrix item. Try doing a PUT call using OAuth 1.0 conenction.

Expected result

The Matrix Sub items should get exported to Magento and Matrix items flow should get completed with success count.

Actual result

The Matrix sub items are not getting exported to Magento and getting errored out. Below is the response we are getting from magento . “{"message":"The signature is invalid. Verify and try again.","trace":"#0 \/var\/www\/html\/enterprise246\/project-enterprise-edition\/vendor\/magento\/framework\/Oauth\/Oauth.php(127): Magento\Framework\Oauth\Oauth->_validateSignature()\n#1 \/var\/www\/html\/enterprise246\/project-enterprise-edition\/vendor\/magento\/module-webapi\/Model\/Authorization\/OauthUserContext.php(81): Magento\Framework\Oauth\Oauth->validateAccessTokenRequest()\n#2 \/var\/www\/html\/enterprise246\/project-ente...“

Additional information

The same scenario is working fine with Token based connection. Facing the issue when the item name has space and special character in it.

Release note

No response

Triage and priority

m2-assistant[bot] commented 5 months ago

Hi @saikalyan425. Thank you for your report. To speed up processing of this issue, make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, Add a comment to the issue:

m2-assistant[bot] commented 5 months ago

Hi @engcom-Delta. Thank you for working on this issue. In order to make sure that issue has enough information and ready for development, please read and check the following instruction: :point_down:

engcom-Delta commented 5 months ago

@magento give me 2.4-develop instance

magento-deployment-service[bot] commented 5 months ago

Hi @engcom-Delta. Thank you for your request. I'm working on Magento instance for you.

magento-deployment-service[bot] commented 5 months ago

Hi @engcom-Delta, here is your Magento Instance: https://9af4fd7d6483b7eba56cde62e9e0f0c8.instances-prod.magento-community.engineering Admin access: https://9af4fd7d6483b7eba56cde62e9e0f0c8.instances-prod.magento-community.engineering/admin_163c Login: aca6f7e9 Password: f7c88beaeb06

engcom-Delta commented 5 months ago

Hi @saikalyan425 ,

Thanks for reporting and collaboration.

Can you let us know to create a Matrix parent item with one sub matrix item , have you used any third party extension?

Thanks,

saikalyan425 commented 5 months ago

Hi team , We are not using any extension here for matrix items. This is happening when the SKU of the items contains space or special characteristics. Below is one of the example where we have faced the issue i.e it has space in the SKU and the next is its endpoint.

"name": "stag mat oauth : stag mat oauth-B"

V1/products/stag%20mat%20oauth%20%3A%20stag%20mat%20oauth-B

FYI, The above scenario works fine with the JWT (Token based connections).

Thank you

On Tue, Jun 4, 2024 at 12:27 PM Madhusmita Sahu @.***> wrote:

Hi @saikalyan425 https://github.com/saikalyan425 ,

Thanks for reporting and collaboration.

We have tried to reproduce this issue on 2.4-develop.

Can you let us know to create a Matrix parent item with one sub matrix item , have you used any third party extension?

Thanks,

— Reply to this email directly, view it on GitHub https://github.com/magento/magento2/issues/38769#issuecomment-2146751707, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMU6NSF5OOLNPMAXWLDBXXLZFVQORAVCNFSM6AAAAABIOISUN6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBWG42TCNZQG4 . You are receiving this because you were mentioned.Message ID: @.***>

--

Sai Kalyan Dusanapudi

Software QA Engineer 2

94911 68559

@.***

[image: image]

[image: image]

[image: linkedin] https://www.linkedin.com/company/celigo-inc/

[image: x] https://twitter.com/celigoinc

[image: facebook] https://www.facebook.com/CeligoInc

[image: youtube] @.***>

[image: image]

[image: image] https://www.celigo.com/ebooks/g2-ipaas-spring-2024-report/

engcom-Delta commented 4 months ago

Hi @saikalyan425 ,

We have verified this issue on 2.4-develop and this is reproducible.

But seems like this is duplicate of https://github.com/magento/magento2/issues/37913 , which is already confirmed in 2.4-develop.

So we are closing this issue as of now. In case you need more updates on this then refer to the above issue.

Thanks,

jorgb90 commented 4 months ago

Already waiting a long time to get this fixed.. https://github.com/magento/magento2/issues/37278