Skip to content

Fix Firebase Functions Emulator usage on Firebase Studio #9204

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
Merged
merged 7 commits into from
Aug 13, 2025
Merged

Fix Firebase Functions Emulator usage on Firebase Studio #9204

merged 7 commits into from
Aug 13, 2025

Conversation

maneesht
Copy link
Contributor

  • Fixed issue where when attempting to load the cookies for the functions emulator, an incorrect URL was used.
  • Fixed issue where requests to the function didn't include the Firebase Studio workstation Cookie

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Aug 12, 2025 •
edited
Loading

🦋 Changeset detected

Latest commit: 556d1be

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@firebase/functions Patch
firebase Patch
@firebase/functions-compat Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@maneesht maneesht marked this pull request as ready for review August 12, 2025 18:23
@maneesht maneesht requested review from a team as code owners August 12, 2025 18:23
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Aug 12, 2025 •
edited
Loading

Size Report 1

Affected Products

  • @firebase/functions

    TypeBase (cc605e7)Merge (acd6049)Diff
    browser13.9 kB14.2 kB+297 B (+2.1%)
    main14.4 kB14.7 kB+302 B (+2.1%)
    module13.9 kB14.2 kB+297 B (+2.1%)

  • bundle

    TypeBase (cc605e7)Merge (acd6049)Diff
    functions (call)38.6 kB38.7 kB+132 B (+0.3%)

  • firebase

    TypeBase (cc605e7)Merge (acd6049)Diff
    firebase-compat.js800 kB800 kB+130 B (+0.0%)
    firebase-functions-compat.js14.7 kB14.8 kB+126 B (+0.9%)
    firebase-functions.js19.4 kB19.5 kB+182 B (+0.9%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/nc3W1xyo4Q.html

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Aug 12, 2025 •
edited
Loading

Size Analysis Report 1

Affected Products

  • @firebase/functions

    • connectFunctionsEmulator

      Size

      TypeBase (cc605e7)Merge (acd6049)Diff
      size2.84 kB2.85 kB+12 B (+0.4%)
      size-with-ext-deps25.0 kB25.1 kB+12 B (+0.0%)

    • getFunctions

      Size

      TypeBase (cc605e7)Merge (acd6049)Diff
      size3.03 kB3.04 kB+12 B (+0.4%)
      size-with-ext-deps32.6 kB32.6 kB+12 B (+0.0%)

    • httpsCallable

      Size

      TypeBase (cc605e7)Merge (acd6049)Diff
      size8.48 kB8.62 kB+141 B (+1.7%)
      size-with-ext-deps26.2 kB26.5 kB+284 B (+1.1%)

      Dependency

      TypeBase (cc605e7)Merge (acd6049)Diff
      functions

      16 dependencies

      _errorForResponse
call
callAtURL
codeForHTTPStatus
createResponseStream
decode
encode
failAfter
httpsCallable
httpsCallable$1
makeAuthHeaders
mapValues
postJSON
registerFunctions
stream
streamAtURL

      17 dependencies

      _errorForResponse
call
callAtURL
codeForHTTPStatus
createResponseStream
decode
encode
failAfter
getCredentials
httpsCallable
httpsCallable$1
makeAuthHeaders
mapValues
postJSON
registerFunctions
stream
streamAtURL

      + getCredentials

      External Dependency

      ModuleBase (cc605e7)Merge (acd6049)Diff
      @firebase/util

      FirebaseError
getModularInstance

      FirebaseError
getModularInstance
isCloudWorkstation

      + isCloudWorkstation

    • httpsCallableFromURL

      Size

      TypeBase (cc605e7)Merge (acd6049)Diff
      size8.37 kB8.51 kB+141 B (+1.7%)
      size-with-ext-deps26.1 kB26.4 kB+284 B (+1.1%)

      Dependency

      TypeBase (cc605e7)Merge (acd6049)Diff
      functions

      14 dependencies

      _errorForResponse
callAtURL
codeForHTTPStatus
createResponseStream
decode
encode
failAfter
httpsCallableFromURL
httpsCallableFromURL$1
makeAuthHeaders
mapValues
postJSON
registerFunctions
streamAtURL

      15 dependencies

      _errorForResponse
callAtURL
codeForHTTPStatus
createResponseStream
decode
encode
failAfter
getCredentials
httpsCallableFromURL
httpsCallableFromURL$1
makeAuthHeaders
mapValues
postJSON
registerFunctions
streamAtURL

      + getCredentials

      External Dependency

      ModuleBase (cc605e7)Merge (acd6049)Diff
      @firebase/util

      FirebaseError
getModularInstance

      FirebaseError
getModularInstance
isCloudWorkstation

      + isCloudWorkstation

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/eVaxED8ATa.html

@maneesht maneesht changed the title Used valid path for pingserver Fix Firebase Functions Emulator usage on Firebase Studio Aug 13, 2025
@maneesht maneesht merged commit 5501791 into main Aug 13, 2025
36 of 38 checks passed
@maneesht maneesht deleted the mtewani/fix-hosting-emulators branch August 13, 2025 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hsubox76 hsubox76 hsubox76 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@maneesht @google-oss-bot @hsubox76