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.

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

@azure/storage-blob runtime TypeError: Cannot read property 'INTERNAL' of undefined #171

Closed
voidfoo opened this issue May 15, 2021 · 2 comments

Comments

@voidfoo
Copy link

voidfoo commented May 15, 2021

import { BlobServiceClient } from "https://cdn.skypack.dev/@azure/storage-blob/";

console.log('hello');

const client = new BlobServiceClient("endpoint");

client.getProperties().then((resp) => {
  console.log(resp);
})

stack trace:

core-tracing.js:368 Uncaught (in promise) TypeError: Cannot read property 'INTERNAL' of undefined
    at core-tracing.js:368
    at BlobServiceClient2.<anonymous> (storage-blob.js:20162)
    at step (tslib.js:221)
    at Object.next (tslib.js:168)
    at tslib.js:154
    at new Promise (<anonymous>)
    at __awaiter2 (tslib.js:136)
    at BlobServiceClient2.getProperties (storage-blob.js:20157)
    at pen.js?key=pen.js-09c6ac50-f140-7d00-144a-744a4d13913f:7

The code is trying to access SpanKind.INTERNAL which is from @ opentelemetry/api

image

import __commonjs_module0 from "/-/@opentelemetry/api@v0.10.2-LVV51xMR0CyWJgXhz3xw/dist=es2020,mode=imports/optimized/@opentelemetry/api.js";
const {TraceFlags} = __commonjs_module0;
const {CanonicalCode} = __commonjs_module0;
const {SpanKind} = __commonjs_module0;
@voidfoo voidfoo changed the title TypeError: Cannot read property 'INTERNAL' of undefined @azure/storage-blob runtime TypeError: Cannot read property 'INTERNAL' of undefined May 15, 2021
@vasco-santos
Copy link

Hello @voidfoo @natemoo-re @drwpow

I ran into the exact same problem when using skypack with a cjs module that depends on a module with multiple builds (cjs, esm, ..)

I nailed this down to a reproducible case that hopefully can get this easily fixed: https://github.com/vasco-santos/cjs-esm-skypack-build-rep

achingbrain added a commit to ipfs/js-ipfs-unixfs that referenced this issue Aug 19, 2021
Switches the codebase to ESM and exports ESM and CJS code. The main goal is to enable ipfs-unixfs to be built on next generation bundlers like skypack: codepen.io/vascosantos/pen/NWjaYqP?editors=0011 This is a problem that has not been fixed in skypack yet. More details: skypackjs/skypack-cdn#171

Given we will need to export ESM and we will, let's get this to circumvent the issue.

We also need to manually change the generated protobuf code due to protobufjs/protobuf.js#1230

BREAKING CHANGE: ./src/dir-sharded is not in the exports map so cannot be imported

Co-authored-by: Alex Potsides <alex@achingbrain.net>
@voidfoo voidfoo closed this as completed Nov 20, 2021
@voidfoo
Copy link
Author

voidfoo commented Nov 20, 2021

The latest @azure/storage-blob is now working fine with skypack

github-actions bot pushed a commit to ipfs/js-ipfs-unixfs that referenced this issue Apr 26, 2022
## ipfs-unixfs-v1.0.0 (2022-04-26)

### ⚠ BREAKING CHANGES

* ./src/dir-sharded is not in the exports map so cannot be imported

Co-authored-by: Alex Potsides <alex@achingbrain.net>
* uses new multiformats stack and takes a blockservice instead of the block api

Co-authored-by: Rod Vagg <rod@vagg.org>
Co-authored-by: achingbrain <alex@achingbrain.net>
* switches to named exports
* types are now included with all unixfs modules
* does not convert input to node Buffers any more, uses Uint8Arrays instead

### Features

* add types ([#114](#114)) ([ca26353](ca26353))

### Bug Fixes

* add pbjs namespace ([#145](#145)) ([dd26b92](dd26b92))
* declare types in .ts files ([#168](#168)) ([76ec6e5](76ec6e5))
* ignore high mode bits passed to constructor ([#53](#53)) ([8e8d83d](8e8d83d))
* ignore undefined values in options ([#173](#173)) ([200dff3](200dff3))
* individual packages can use npm 6 ([#167](#167)) ([2b429cc](2b429cc))
* publish with types in package.json ([#166](#166)) ([0318c98](0318c98))
* remove node globals ([#52](#52)) ([5414412](5414412))
* replace node buffers with uint8arrays ([#69](#69)) ([8a5aed2](8a5aed2)), closes [#66](#66)
* types with ipjs build ([#165](#165)) ([fea85b5](fea85b5))
* use @ipld/dag-pb instead of ipld-dag-pb ([#116](#116)) ([bab1985](bab1985))

### Trivial Changes

* add travis file and configure build scripts ([5a25c87](5a25c87))
* consolidate .gitignore files ([b05e468](b05e468))
* declare interface types in .d.ts file ([#122](#122)) ([eaa8449](eaa8449))
* dep updates ([cf9480b](cf9480b))
* **deps-dev:** bump aegir from 26.0.0 to 28.1.0 ([#86](#86)) ([87541c7](87541c7))
* **deps-dev:** bump aegir from 28.2.0 to 29.2.2 ([#101](#101)) ([010ab47](010ab47))
* exclude docs and tests from npm package ([63b8ba0](63b8ba0))
* move files into packages folder ([943be9d](943be9d))
* publish ([5203595](5203595))
* publish ([0f9092e](0f9092e))
* publish ([2713329](2713329))
* publish ([35e2059](35e2059))
* publish ([137a4ad](137a4ad))
* publish ([f173850](f173850))
* publish ([2ea467f](2ea467f))
* publish ([dedbd82](dedbd82))
* publish ([dc2d400](dc2d400))
* publish ([27d57df](27d57df))
* publish ([5ccac2f](5ccac2f))
* publish ([172548b](172548b))
* publish ([9a2b5f2](9a2b5f2))
* publish ([e57ba16](e57ba16))
* publish ([9e8f077](9e8f077))
* publish ([22e29bb](22e29bb))
* publish ([dabbb48](dabbb48))
* publish ([32e5165](32e5165))
* publish ([5d3f4bd](5d3f4bd))
* publish ([49c8c54](49c8c54))
* publish ([db2c878](db2c878))
* publish ([9237250](9237250))
* remove changes from readme ([7c727ef](7c727ef))
* remove node buffers from runtime code ([#66](#66)) ([db60a42](db60a42))
* remove redundant test files ([3078608](3078608))
* small readme change ([f45436c](f45436c))
* swap to prereleaseOnly ([efb01ac](efb01ac))
* switch to auto-release ([#208](#208)) ([99386e6](99386e6))
* switch to ESM ([#161](#161)) ([819267f](819267f)), closes [skypackjs/skypack-cdn#171](skypackjs/skypack-cdn#171)
* tighten up input types ([#133](#133)) ([47f295b](47f295b))
* update build scripts ([37d96ee](37d96ee))
* update deps ([#144](#144)) ([f5f5fe4](f5f5fe4))
* update deps ([#78](#78)) ([2bf5d07](2bf5d07))
* update lockfiles ([9d11252](9d11252))
* update package.json scripts and readmes ([bda3717](bda3717))
* update readme ([a012f22](a012f22))
* update readme ([7f93da1](7f93da1))
* update readmes ([#188](#188)) ([273a141](273a141))
* upgrade deps ([3a43e92](3a43e92))
* use npm 7 workspaces instead of lerna bootstrap ([#120](#120)) ([1ceb097](1ceb097))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants