-
Notifications
You must be signed in to change notification settings - Fork 979
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
firebase-tools serve makes an error #1815
Comments
This issue does not have all the information required by the template. Looks like you forgot to fill out some sections. Please update the issue with more information. |
Thanks for filing this! Could you show the source code for the function
you're calling?
…On Tue, Nov 19, 2019, 2:21 AM Google Open Source Bot < ***@***.***> wrote:
This issue does not have all the information required by the template.
Looks like you forgot to fill out some sections. Please update the issue
with more information.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#1815?email_source=notifications&email_token=ACATB2VV7USTP75TIXMYFX3QUO43LA5CNFSM4JPBCNG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEENU4XY#issuecomment-555437663>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACATB2XFOWJFQF45ZDBVYELQUO43LANCNFSM4JPBCNGQ>
.
|
+1 Same issue with latest version (7.8.0). Second call to the function throws this error. Just rolled back to 7.7.0 and it works fine. |
@osadchiynikita can you show me the simplest function you have that reproduces this error? |
@samtstern I just started to set up new project, so I was testing with the simple hello function
With the next firebase.json config
When I'm calling function first time (open in browser), it executes correctly, but for the next call (page reload) server throws "Address in use" error P.S. Thank you for the fast response |
@osadchiynikita I was able to reproduce this, thank you for your help! Definitely downgrade to |
@samtstern Great! Thank you |
Ok it happened during this PR: So it's my fault (obviously). Now need to dig. |
@osadchiynikita @samtstern Thank you for all! I was worried about this, because I'm developing based on firebase. |
I figured this out and a fix will be merged soon. In the meantime everyone
here should use 7.7.0.
…On Tue, Nov 19, 2019, 5:35 PM Yiggdrasil ***@***.***> wrote:
@osadchiynikita <https://github.com/osadchiynikita> @samtstern
<https://github.com/samtstern> Thank you for all! I was worried about
this, because I'm developing based on firebase.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1815?email_source=notifications&email_token=ACATB2WMJVA7THMAYKEZ2HLQUSH4TA5CNFSM4JPBCNG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEQMDGI#issuecomment-555794841>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACATB2XFR5DYGMLT236JLA3QUSH4TANCNFSM4JPBCNGQ>
.
|
omg @parsher this was driving me nuts all yesterday, I was convinced it was something I was doing while configuring my Vue project to use the emulators in dev and live in production. Downgraded to 7.7.0 and it works fine. I didn't twig that I had updated firebase-tools yesterday morning, just before I started having issues. :| @samtstern I'll only blame you for an hour or so ha ha. |
For everyone affected by this: version |
@samtstern Thank you for your fast release! |
[REQUIRED] Environment info
**firebase-tools: 7.8.0 (the latest)
**Platform: macOS
[REQUIRED] Test case
firebase functions
[REQUIRED] Steps to reproduce
firebase functions + express get / post methods
request multiple get requests, it makes errors.
it is only made on firebase-tools 7.8.0.
if I run the same codes on firebase-tools 7.7.0., it works fine.
[REQUIRED] Expected behavior
If I request multiple requests, it accepts well and perform.
[REQUIRED] Actual behavior
it makes an error
The text was updated successfully, but these errors were encountered: