Add "content-type" to connheaders()

[jchvz]

Neo4j now requires a Content Type header in all API requests. I added it here by changing the connheaders() function.

"content-type": "application/json"

function connheaders(c::Connection)
  headers = Dict(
    "Accept" => "application/json; charset=UTF-8",
    "content-type" => "application/json",
    "Host" => "$(c.host):$(c.port)")
  if c.user != "" && c.password != ""
    payload = "$(c.user):$(c.password)" |> base64encode
    headers["Authorization"] = "Basic $(payload)"
  end
  headers
end

For me, none of the functions were working until I added this. I am using Neo4j 4.3.6.

Thanks for your work, I am liking the automatic DataFrame feature.

[jchvz]

Hi @glesica ,

I know this library isn't maintained anymore, but will this update be released? I believe it solves the connection problems that others were describing.

It works on Neo4j 4.3.6 for me:

I think it would be valuable to have the version that comes down with using Neo4j work again. Maybe it just needs for travis to be turned back on again; unfortunately I don't really understand the release process.

Thanks

[glesica]

I haven't done a Julia release in quite a while. As I recall, there's a repo you have to PR into to update the version, but they do a bunch of checks on the package before they allow it, and I'm not sure this package would pass muster any more. I'm unwilling to put much effort into it, but if someone wants to figure out how to do it and do the legwork, go for it.