-
Notifications
You must be signed in to change notification settings - Fork 16
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
Upgrade to Julia 0.7; switch to HTTP.jl #20
Conversation
this is going to fix |
Thanks for working on this! |
Looks like the Travis and AppVeyor CI files need to be updated for the latest versions. |
src/session.jl
Outdated
@@ -137,19 +141,39 @@ function JWS(credentials::JSONCredentials, claimset::JWTClaimSet, header::JWTHea | |||
"$payload.$signature" | |||
end | |||
|
|||
function token(credentials::JSONCredentials, scopes::AbstractVector{<: AbstractString}) | |||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking at #21 - HTTP.URIs.escapeuri
should cover this
@joshbode I just updated the travis and appveyor and other code, but still have some test error of storage api. I got some difficulty to locate the error code, do you have any idea?
|
here is the full error message:
|
Looks like the issue is:
|
I have fixed that error, but get some more.
|
@samoconnor @quinnj There are some errors related with HTTP. It probably is a bug of this package rather than HTTP. |
The stack trace says you're calling https://juliaweb.github.io/HTTP.jl/latest/index.html#HTTP.Messages.header https://juliaweb.github.io/HTTP.jl/latest/index.html#HTTP.Messages.hasheader |
I have fixed the bugs, tests passed. I have only tested with Storage backend and KeyStore functions though. |
@StefanKarpinski I would like to register this upgrade to METADATA.jl using attobot, but I can not set it up since I do not have the permission. Could you please help to set it up or just give me the permission? There is an issue from attobot complaining the tagging link is not correct, since it was migrated from @joshbode 's repo. |
I also can not edit the documentation link which is invalid and outdated. |
Thanks for all your effort @jingpengw! I also am not able to edit that link for some reason... |
Julia 0.7 is pretty nice, let's upgrade!
I also switched to HTTP.jl since it is update to Julia 0.7