-
Notifications
You must be signed in to change notification settings - Fork 762
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
Unable to upload crawler-beans.cxml with curl #282
Comments
Thanks for the bug report. Confirming I can reproduce this. It's most likely a regression from the Restlet 2 upgrade (#276). I will debug it and aim to have a fix shortly. |
I have a partial fix for the Method Not Allow error in 5612fa2 however unfortunately that in turn exposed another problem with PUT under Restlet 2. Surprisingly (to me at least) it appears Restlet actually removes the file extension from the filename and then replaces it: This means PUT crawler-beans.cxml ends up creating a file incorrectly named crawler-beans.xml. I will continue working on this tomorrow as it's rather late here now. @lpla in the meantime I suggest using the 3.4.0-20190418 release rather than master as it was prior to the restlet2 upgrade. |
This is very reminiscent of https://webarchive.jira.com/browse/HER-1907, filed in 2011:
|
I guess Restlet's philosophy kind of makes sense for updating static website content in multiple languages and such, although even for that I'm sceptical. It's also inconsistent in that other methods like DELETE don't manipulate the file extension. For Heritrix's use case of providing API access to the jobs directory I can't imagine any scenario where we'd want it do this. So I'm proposing we override PUT to disable the file extension manipulation entirely. |
Hi. We are trying to integrate
heritrix
inbitextor
and we are having issues with REST API commands usingcurl
. We are trying to automate the whole crawling process but after creating a job, uploading the config filecrawler-beans.cxml
is not working. Paste command and response:The text was updated successfully, but these errors were encountered: