-
Notifications
You must be signed in to change notification settings - Fork 9
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
Information about AWS ec2 instance metadata credentials #38
Comments
I've corrected the title and body, we are not using OIDC but EC2 metadata credentials. |
Most of the logic in this repo stems from https://github.com/christophgysin/aws-sdk-js-v3/blob/deno/scripts/generate-clients/copy-to-deno.js. Did you get an older version to work using esm.sh? This port is mostly abandoned after aws-sdk started delegating a lot of the work to a new depdendency |
@christophgysin What do you mean by "a port"? I thought I just constructed this URL: https://esm.sh/@aws-sdk/[email protected], and I seem to be able to I started looking into it when we had an issue bundling with Download https://jspm.dev/npm:@aws-sdk/util-utf8-browser@3!cjs
Download https://jspm.dev/npm:[email protected]!cjs
Download https://jspm.dev/npm:@aws-sdk/[email protected]!cjs
Download https://deno.land/[email protected]/node/querystring.ts
error: Module not found "https://jspm.dev/npm:@aws-sdk/[email protected]".
at https://jspm.dev/@aws-sdk/hash-node:4:8
The command '/bin/sh -c deno cache lib/docker.ts' returned a non-zero code: 1 I also don't know if that's normal but the last tag on this repo is on Can you advise about what is the right thing to do (future-proof) with aws-ask for deno? Cheers. |
There is some misunderstanding here. The original source is at https://github.com/aws/aws-sdk-js-v3. From there, every package is published to npm, e.g. https://www.npmjs.com/package/@aws-sdk/client-s3. esm.sh is transpiling automatically from npm, and provides the result at https://esm.sh/@aws-sdk/client-s3. This repo is a fork of https://github.com/aws/aws-sdk-js-v3, that translates the code to deno APIs, and publishes the result to https://deno.land/x/aws_sdk. Unfortunately aws-sdk started to split out low level functionality to aws-crt, see #28. If you're willing to help getting this fork to work with aws-crt (or find a way around it), let me know. Otherwise you can ask esm.sh for help with their autogenerated port, but I can imagine most functionality won't work out of the box, and they are more focused on translating as many simple packages as possible instead of perfecting a single one (aws-sdk). Or go knock on AWS doors to allow directly maintaining a deno port in the upstream repo, similar to how they currently support node and browser. |
Hey @christophgysin, thanks for clarifying all this. |
Hey @christophgysin,
Can you clarify what you meant by "this port", are you talking about this fork or any I'm after what is the most reliable and future-proof usage for the AWD SDK with Deno, as AWS does not provide an official port. We are porting some scripts in our CI and we want this part to be as stable as possible. |
This fork is a port of the aws-sdk-v3 to deno. If I'm referring to "this port", I'm referring to this repository we are discussing in. There is currently no reliable version of aws-sdk for deno. |
Thanks for clarifying, I created another issue there: |
Hi,
This is not a feature request, but it wasn't a bug either.
In case anyone has some insights, it would be greatly appreciated.
I just noticed that using an
esm.sh
based url:https://esm.sh/@aws-sdk/[email protected]
Instead of:
https://deno.land/x/[email protected]/client-s3/mod.ts
The instance metadata credentials are not being picked up:
Similar issue
As with the
deno.land/x/aws_sdk
module I did not have to set up anything special in terms of credentials, I don't see why I would need to set up a credentials provider manually.Has something been done in this repo?
Any clue of what could be the issue?
The text was updated successfully, but these errors were encountered: