https://discord.cloudflare.com logo
Join Discord
Powered by
# r2
  • d
    Yeah Get the string from that thing Then I will help you find out what it's in
  • j
  • d
    Oh
  • d
    Love how cf code are open
  • j
    yes! these types in particular are great
  • j
    ouch - no 
    startAfter
    in list options (bindings), has this been reported yet?
  • j
    answering my own question - it appears to be hex. Validation error on garbage input: 
    MD5 is 32 hex characters
  • e
    Did you ever work this out? Am experiencing the same
  • v
    Yeah. We're in feature freeze though so we're only fixing bugs.
  • e
    i also tried debugging sharex S3 support (not custom uploader like others) and for some reason it gets ```xml InvalidRequestPlease use AWS4-HMAC-SHA256```despite using AWS4-HMAC-SHA256 for the request (something like 
    AWS4-HMAC-SHA256 Credential=credhere/20220428/world/s3/aws4_request,SignedHeaders=content-length;content-type;host;x-amz-content-sha256;x-amz-date;x-amz-storage-class,Signature=signature
    )
  • v
    missing x-amz-content-sha256?
  • e
    thats a header right? I see it mentioned here
  • e
    (i cloned the sharex source code to look and properly debug it, lol)
  • e
    does that header need to be mentioned in the signature as well? cos i noticed it wasnt
  • v
    "invalid sigv4 algorithm AWS4-HMAC-SHA256"
  • v
    That usually means that the format of the signature line is incorrect
  • v
    It's possible we're more finnicky than S3
  • e
    yeah, sharex works fine with a lot of other S3 providers, but i wouldnt be surprised either if its not following spec exactly
  • e
    the signature line should be the one above, i notice it does have 
    x-amz-content-sha256
    and the header for that should be there too
  • v
    Uggh. We require a space
  • v
    Copy code
    // format is: AWS4-HMAC-SHA256 Credential=, SignedHeaders=, Signature=
  • v
    We split on 
    ,
    instead of split on 
    ,
    and trim
  • v
    I'll raise it internally
  • e
    Oh, haha
  • e
    Ahaha yeah it works if I modify the ShareX source to use 
    ", "
    , thank you for the help and for raising it!
  • v
    If my analysis was correct we should have that fixed tomorrow. The auth expert went home so no review tonight
  • v
    I’m extremely happy I didn’t have to implement sigv4. It’s deceptively complex
  • i
    I feel that every time I used it
  • i
    Thankfully most of the libraries take care of it for me.
  • e
    Now the only mystery to me is why rclone 401s on upload but not list 🤔 but I don't know if I want to dissect the rclone source as well
1...697071...1050Latest