Skip to content

Incorrect Request URL with Form Key on the product import. #5154

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

Closed
maksymhopei opened this issue Jun 22, 2016 · 3 comments
Closed

Incorrect Request URL with Form Key on the product import. #5154

maksymhopei opened this issue Jun 22, 2016 · 3 comments
Labels
bug report Component: ImportExport Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is not valid Gate 1 Failed. Automatic verification of issue format is failed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release

Comments

@maksymhopei
Copy link

maksymhopei commented Jun 22, 2016

Steps to reproduce

  1. Install Magento from develop branch.
  2. Go to Admin Panel
  3. Go to System -> Import.
  4. Select "Products" in Entity Type field.
  5. Set: Import Behavior: "Add/Update", select valid file to import.
  6. Click "Check Data", wait for validation to complete.
  7. Click "Import".

Expected result

  1. The request is sent to http://obo-demo.testing.cgi-labs.de/admin/admin/import/start/key/[some-key]/?form_key=[form-key] OR
  2. The request URL doesn't contain form key as form key is present in the POST payload.

Actual result

  1. Request URL is: http://obo-demo.testing.cgi-labs.de/admin/admin/import/start/key/[some_key]/&form_key=[form-key] (note the "&" instead of "?"). Form key is also in the payload.
@veloraven veloraven self-assigned this Jun 23, 2016
@veloraven veloraven removed their assignment Jun 23, 2016
@shiftedreality
Copy link
Member

Hi @radio

Thank you for your submission.
We've created internal ticket MAGETWO-58250 to fix this issue.

@shiftedreality shiftedreality added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Sep 9, 2016
@magento-engcom-team magento-engcom-team added bug report Issue: Format is not valid Gate 1 Failed. Automatic verification of issue format is failed Component: ImportExport labels Sep 11, 2017
@magento-engcom-team magento-engcom-team added Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed 2.1.x Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release labels Oct 13, 2017
@magento-engcom-team
Copy link
Contributor

@radio, thank you for your report.
We've created internal ticket(s) MAGETWO-58250 to track progress on the issue.

@piotrekkaminski
Copy link
Contributor

This issue was moved to magento-engcom/import-export-improvements#58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Component: ImportExport Issue: Clear Description Gate 2 Passed. Manual verification of the issue description passed Issue: Confirmed Gate 3 Passed. Manual verification of the issue completed. Issue is confirmed Issue: Format is not valid Gate 1 Failed. Automatic verification of issue format is failed Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Reproduced on 2.3.x The issue has been reproduced on latest 2.3 release
Projects
None yet
Development

No branches or pull requests

6 participants