search

yorkittran / google-scraper

0 stars 0 forks source link

[Bug] CSV upload fails #11

Open olivierobert opened 1 year ago

olivierobert commented 1 year ago

Issue

Upon uploading a CSV file, the UI shows a success message:

2566-03-22 15 49 51

However, the web server log shows an authorization issue:

image

As a result, I have not been able to verify the scraping results yet 😅

yorkittran commented 1 year ago

@olivierobert In this PR I just fixed the unpermitted parameter. About why it still does not work, I think you should run your Sidekiq by bundle exec sidekiq command. I will add this to README later 😭

olivierobert commented 1 year ago

I was also surprised by the usage of strong params:

params.require(:file)
params.permit(:file)

As the standard usage is nested params 😅

Previously nothing was created in the database but noted on the need to run Sidekiq.

Note You might want to look into using https://github.com/ddollar/foreman to run all required applications with the application. Here is an example from Nimble's Rails template