by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
Jonathan A Rees
@jar398
Probably sufficient to search for /service/. Thanks
Jeremy Rice
@JRice
Sorry, @jar398 , I don't see any errors in the logs on production with /service/
OOC, do you have to be logged in? I sometimes get that error on the admin pages when I forget...
I ask because I see a lot of successful calls to that page, e.g.:
method=POST path=/service/cypher format=json controller=Service::CypherController action=command status=200 duration=9815.35 view=62.57 db=0.61
Jonathan A Rees
@jar398
uh maybe that's the wrong search string. Some of the requests have worked (those from the traits dumping script), it's a random URL that's failing with a 500. The bug is not so much the failure as the fact that it was a 500 with no useful error message. All errors are supposed to generate 4xx codes.
Jeremy Rice
@JRice
Since I'm looking for errors (Sorry, still don't see any related to Cypher!), I am seeing a lot that look like this:
ActionController::RoutingError (No route matches [GET] "/pages/2315/"https:/web.archive.org/web/20131024173037/http:/edutube.org/video/coconut-shelter-evidence-tool-use-octopuses""):
ActionController::RoutingError (No route matches [GET] "/pages/2315/"https:/www.theguardian.com/world/2016/apr/13/the-great-escape-inky-the-octopus-legs-it-to-freedom-from-new-zealand-aquarium"):
...Which strikes me as ... odd...
...Maybe it's just someone looking for a security hole, but... thought I'd mention it in case someone has an idea.
Jonathan A Rees
@jar398
The particular case is: https://eol.org/service/cypher?query=%22MATCH%20(n:Trait)%20RETURN%20n%20LIMIT%201;%22
which might be erroneous, but should at least give a 4xx instead of a 5xx
Jeremy Rice
@JRice
Ahhh, thanks. I was able to trigger that error:
Jonathan A Rees
@jar398
that would be entered in a web browser. The cypher-form version of this works
Jeremy Rice
@JRice
method=GET path=/service/cypher format=html controller=Service::CypherController action=query status=500 error='Neography::SyntaxException: Invalid input '"': expected <init> (line 1, column 1 (offset: 0))
""MATCH (n:Trait) RETURN n LIMIT 1"
 ^' duration=8.30 view=0.00 db=0.69

Neography::SyntaxException (Invalid input '"': expected <init> (line 1, column 1 (offset: 0))
""MATCH (n:Trait) RETURN n LIMIT 1"
 ^):

app/models/trait_bank.rb:32:in `query'
app/controllers/service/cypher_controller.rb:77:in `cypher_command'
app/controllers/service/cypher_controller.rb:36:in `query'
...Looks like perhaps the controller could do a better job of catching that kind of error?
Jonathan A Rees
@jar398
Weird, that means the code is missing a rescue. Thanks
JRice @JRice nods
Jonathan A Rees
@jar398
Yep that's it
Thanks much
Jeremy Rice
@JRice
Thanks for catching it.
...errr... "catching" ...
If only this were Python, I might be "funny"...
Jeremy Rice
@JRice
Might be blip in production search, restarting ES
Jeremy Rice
@JRice
Okay! Survey was ... useful. I should maybe make this a regularly-scheduled thing, like once a year or so.
Jen Hammock
@jhammock
good to know we have an inventory! Did you get my comment from Eli? It seemed like the context in which to pass it on
Jeremy Rice
@JRice
Yes, noted. Thanks.
Jen Hammock
@jhammock
:+1: if we do need to go shopping, feel free to propose it to Tom. I doubt my getting in the middle would be helpful
Jeremy Rice
@JRice
Noted, thanks.
Jen Hammock
@jhammock
How is that new metadata migration job? It seemed to make some progress- polished off the last 2896124 Source metadata- but I think it may have given up after that
Jeremy Rice
@JRice
Still running.
Jen Hammock
@jhammock
Cool, thanks
Jeremy Rice
@JRice
Last notice:
copied 10000 records
1882087 records remaining
...Which is ... kinda crazy. I'm at a loss WRT what happened there.
Jen Hammock
@jhammock
well, the records look ok- that is, the new properties have appeared where the metadata nodes were removed. And we should only need to do this once...
Jeremy Rice
@JRice
Right. I'm just still baffled that the script was convinced it was done on Saturday.
Jen Hammock
@jhammock
Yeah, I don’t think the beta run had issues like this
Jeremy Rice
@JRice
Maybe the neo4j DB borked at the script just thought "well, maybe there are 0 left if I can't contact the server" or something.
Certainly not!
Michael Vitale
@mvitale
should I have a look at the script? maybe it's messed up..
Jeremy Rice
@JRice
Let's say "if it doesn't really work this time, yes."
Michael Vitale
@mvitale
:+1:
Jeremy Rice
@JRice
Not a script worth "maintaining," sooo....
Michael Vitale
@mvitale
right
Jeremy Rice
@JRice
I'll be out sick today. Sorry: haven't felt well for a few days and today it's a bit worse. :S
Jen Hammock
@jhammock
drink fluids!
Jeremy Rice
@JRice
I always do! :)
Jeremy Rice
@JRice
(Just fair warning: I am only feeling worse this evening. I'll let you know tomorrow, but I wouldn't hold my breath that this point.) :(
Jen Hammock
@jhammock
Gah. Good luck figuring out your pending errands :(