Allow GET requests to harvester.refresh

ckan / ckanext-harvest

Remote harvesting extension for CKAN

130 stars 203 forks source link

Allow GET requests to harvester.refresh #452

Closed bzar closed 3 years ago

bzar commented 3 years ago

The "Reharvest" button in harvest source admin view returns a 404 because it performs a GET request and the blueprint doesn't cover them. Added GET to allowd methods to fix this.

Financed by Finland's open data portal opendata.fi. Find all Finnish open data at https://www.opendata.fi/en. The Service is provided by the Digital and Population Data Services Agency (https://dvv.fi/en/).

bzar commented 3 years ago

The same issue seems to happen with clear

amercader commented 3 years ago

I was wondering why this hasn't come up before as it seems pretty obvious and it's because the Reharvest and Clear buttons fire a modal dialog. When confirming it we must be sending a POST request with js instead of GET. It makes sense to support both.

@bzar can you apply the same fix to the Clear endpoint?