Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[remerge] A more spec-compliant/resilient OCI distribution implementation #318

Merged
merged 18 commits into from
Dec 7, 2022
Merged

[remerge] A more spec-compliant/resilient OCI distribution implementation #318

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
68d50ca
setup reference implementation registry for testing
joshspicer Dec 2, 2022
ebfc4c3
uncomment
joshspicer Dec 2, 2022
68ae246
revert plainHTTP defaulting to true
joshspicer Dec 2, 2022
1945816
mode for plain http when targetting localhost
joshspicer Dec 2, 2022
2701867
print only when plain HTTP
joshspicer Dec 2, 2022
c6f673c
surface JSON errors better
joshspicer Dec 2, 2022
ec68d10
refactor out reading in tgz Buffer to only do that work once
joshspicer Dec 2, 2022
1d5ed62
generate a postUploadSessionId for each blob that needs to be uploaded
joshspicer Dec 2, 2022
20d7774
update auth contract change in utils file
joshspicer Dec 2, 2022
28dd163
factor out containerFeaturesOCIPush test to its own matrix run
joshspicer Dec 2, 2022
de93dad
clarifying comment
joshspicer Dec 2, 2022
b27f9fc
add comments clarifying auth
joshspicer Dec 2, 2022
56ceefc
allow localhost past the domain name check
joshspicer Dec 2, 2022
866ce3e
add 'info tags' and 'info manifest' commands'
joshspicer Dec 2, 2022
227a85e
use new info commands in test
joshspicer Dec 3, 2022
f80ba58
typecheck
joshspicer Dec 3, 2022
cdd92f5
clarify error case of getManifest()
joshspicer Dec 3, 2022
fd4de3f
Apply suggestions from code review
joshspicer Dec 6, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
update auth contract change in utils file
  • Loading branch information
@joshspicer
joshspicer committed Dec 2, 2022
commit 20d77741209874e2fc92de95f7dfc57e55ffbccd
9 changes: 6 additions & 3 deletions src/spec-configuration/containerCollectionsOCI.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -217,6 +217,7 @@ export async function getManifest(output: Log, env: NodeJS.ProcessEnv, url: stri

return manifest;
} catch (e) {
output.write(`(!) Failed to fetch manifest: ${e}`, LogLevel.Error);
return undefined;
}
}
Expand All @@ -243,8 +244,7 @@ async function getBasicAuthCredential(output: Log, registry: string, env: NodeJS
return Buffer.from(userToken).toString('base64');
}

// Error
output.write(`No authentication credentials found for registry '${registry}'.`, LogLevel.Error);
output.write(`No authentication credentials found for registry '${registry}'.`, LogLevel.Warning);
return undefined;
}

Expand All @@ -263,7 +263,10 @@ async function generateScopeTokenCredential(output: Log, registry: string, ociRe
basicAuthTokenBase64 = await getBasicAuthCredential(output, registry, env);
}

headers['authorization'] = `Basic ${basicAuthTokenBase64}`;
if (basicAuthTokenBase64) {
headers['authorization'] = `Basic ${basicAuthTokenBase64}`;
}


const authServer = registry === 'docker.io' ? 'auth.docker.io' : registry;
const registryServer = registry === 'docker.io' ? 'registry.docker.io' : registry;
Expand Down
2 changes: 1 addition & 1 deletion src/spec-node/utils.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,7 @@ export async function inspectImageInRegistry(output: Log, name: string, authToke
};

if (auth) {
headers['authorization'] = `Bearer ${auth}`;
headers['authorization'] = auth;
}

const options = {
Expand Down