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

Signature parameter no longer send #5

Open
marekgach opened this issue Jun 2, 2016 · 5 comments
Open

Signature parameter no longer send #5

marekgach opened this issue Jun 2, 2016 · 5 comments

Comments

@marekgach
Copy link

My code was working properly and somehow Shopify stopped to send signature parameter. This will lead to:

Notice: Undefined index: signature in **/vendor/phpish/shopify/shopify.php on line 21

in function is_valid_request while validating request. As stated here https://help.shopify.com/api/guides/authentication/oauth#verification says: The signature attribute is deprecated due to vulnerabilities in how the signature is generated.

So it is possible they removed it in previous days from Shopify? Any suggestions?

@archimedesign
Copy link

Not sure if that's related but I couldn't access one of my shops on 26th
May and here's the reply from their support, maybe they were changing
something at the time:
"We experienced a brief outage earlier in the day, however the team has
resolved the issue."
On 2 Jun 2016 16:37, "Marek Gach" [email protected] wrote:

My code was working properly and somehow Shopify stopped to send signature
parameter. This will lead to:

Notice: Undefined index: signature in **/vendor/phpish/shopify/shopify.php
on line 21

in function is_valid_request while validating request. As stated here
https://help.shopify.com/api/guides/authentication/oauth#verification
says: The signature attribute is deprecated due to vulnerabilities in how
the signature is generated.

So it is possible they removed it in previous days from Shopify? Any
suggestions?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#5, or mute the thread
https://github.com/notifications/unsubscribe/AGe91kJ9YYpovixyzZzJGja-KSl6n_Foks5qHvidgaJpZM4IssFN
.

@marekgach
Copy link
Author

MD5 signature is no longer supported since 1st June. So it may be fixed like this: https://github.com/hsleewis/shopify/commit/fced4542010fcc566472fc6fefb36e195872fd49

However even this solution is not working for me when I verify the request for plugin preferences page. Because this request has some different parameters and SHA hash does not match at all.

@hellozach
Copy link

@marekgach - wrap the http_build_query() in urldecode()

See here: #4 (comment)

@ajrana000
Copy link

Please check this : #6

It will solve this issue.

@LukeTowers
Copy link

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

No branches or pull requests

5 participants