-
Notifications
You must be signed in to change notification settings - Fork 590
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
Fix blobs request #350
Fix blobs request #350
Conversation
0d98697
to
df2ae3e
Compare
df2ae3e
to
b796e33
Compare
Hey @Robert-Steiner! Thanks for this! Could you also add a test to ensure this behavior is good? Thanks :) |
Hey @ParthSareen, |
cfd9bde
to
a9a6b4e
Compare
Hey @Robert-Steiner could you fix the conflicts? Rest looks good! Will get this in when fixed. Sorry for the delay! |
Signed-off-by: Robert Steiner <robert@flower.ai>
a9a6b4e
to
75792c5
Compare
Hey @ParthSareen I resolved the merge conflict :) |
I don't know if something is wrong on my end but I don't think I see your changes anymore? LOL @Robert-Steiner |
@ParthSareen you are right! It was fixed in #420 |
Oops I think we could've still gotten the tests in haha - I'll get it in! Appreciate the work! |
The blobs request fails with
invalid digest format
becausesha256:
is added twice in lines 557 and 560.This PR removes the extra
sha256:
in line 560.