-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Could not resolve "@apollo/usage-reporting-protobuf" #7159
Comments
We'll need more details in order to reproduce this: a full set of instructions that lets us see it on our machine. (Perhaps starting with The line of code from |
I have a small example here https://github.com/gersongams/apollo-serverless-boilerplate |
not sure if its because of the esbuild bundle |
@gersongams What commands should I run after cloning? Ideally one that doesn't require setting up AWS credentials? |
I am not a serverless expert but |
I hit this (or a very similar issue) now as well during the upgrade from 4.3.2 to 4.4.1:
Sadly, at the moment I can only reproduce it in the deploy environment (Azure functions), see JabRef/JabRefOnline#1803. Maybe a result of #7411 @trevor-scheer ? In fact downgrading to 4.4.0 solved this problem. |
Thanks for reporting @tobiasdiez, I'll investigate this morning. |
@tobiasdiez that error message looks a little strange to me, is there a reasonable explanation for the missing slash in the file path (between
Edit: nvm, I inspected the source of the comment and it's escaping the @ 🤦 |
@tobiasdiez where can I see that error? I found a similar error in the logs on the PR you linked, but it wasn't the same and it didn't point to any module resolution error. I know you said you don't have a reproduction right now. I tried to reproduce in a simple project and couldn't. If you do get a chance to investigate a bit further that would be helpful - I'm not sure how to be helpful at this point. I've also looked at the published packages and nothing seems obviously off. |
Thanks @trevor-scheer for having a look at this. Yes, the logs are containing only the public-facing error message, which hides all the details. The full error is only visible in the azure log console. I'll try to reproduce it locally, without the deployment to azure. Do you have any idea where this error is coming from? |
@tobiasdiez if you can take a look into the dependency tree / lockfile that might be helpful, after that I'm really not sure.
|
I tried to reproduce it locally, but strangely enough it is working even in azure's local emulator. But your idea with the dependency tree was right. Before I had
(note the two different versions of protobuf), which after yarn dedup resulted in
Which now worked. Yeah! 🥳 Thanks for your help! |
Glad that worked! |
I'm having issues with this version of apollo, I followed this docs
Version:
"@apollo/server": "^4.1.1",
"@as-integrations/aws-lambda": "^1.0.1",
The text was updated successfully, but these errors were encountered: