Skip to content

feat(functions): Allow configuration of Functions Emulator Origin#2017

Merged
jamesdaniels merged 5 commits intomasterfrom
functions_emulator
May 20, 2019
Merged

feat(functions): Allow configuration of Functions Emulator Origin#2017
jamesdaniels merged 5 commits intomasterfrom
functions_emulator

Conversation

@jamesdaniels
Copy link
Copy Markdown
Member

@jamesdaniels jamesdaniels commented Feb 22, 2019

Checklist

  • Issue number for this PR: #nnn (required)
  • Docs included?: (yes/no; required for all API/functional changes)
  • Test units included?: (yes/no; required)
  • In a clean directory, yarn install, yarn test run successfully? (yes/no; required)

Description

Allow AngularFireFunctions to point to the Cloud Functions Emulator. This increases minimum requirement of Firebase to 5.5.7.

Code sample

Point callable Functions to the Cloud Function emulator by adding the FunctionsEmulatorOriginToken to the providers section of your NgModule.

import { NgModule } from '@angular/core';
import { AngularFireFunctionsModule, FunctionsEmulatorOriginToken } from '@angular/fire/functions';
@NgModule({
  imports: [
    ...
    AngularFireFunctionsModule,
    ...
  ],
  ...
  providers: [
   { provide: FunctionsEmulatorOriginToken, useValue: 'http://localhost:5005' }
  ]
})
export class AppModule {}

@jamesdaniels jamesdaniels added this to the 5.2.0 milestone Feb 22, 2019
@jamesdaniels jamesdaniels assigned davideast and unassigned davideast Feb 23, 2019
@jamesdaniels
Copy link
Copy Markdown
Member Author

jamesdaniels commented Feb 23, 2019

@davideast Ready for review. Don't merge, if approved; as I'm targeting this for the 5.2 release.

@jamesdaniels jamesdaniels changed the title Functions emulator Allow configuration of Functions Emulator Origin Feb 25, 2019
@Splaktar
Copy link
Copy Markdown
Contributor

Splaktar commented Mar 15, 2019

This would be nice to have! Looks like it needs a rebase tho.

@jorroll
Copy link
Copy Markdown

jorroll commented May 10, 2019

@jamesdaniels currently, if you are using a custom domain with firebase functions (via firebase hosting) I don't believe @angular/fire has any way of specifying the origin for callable functions requests (I believe you are forced to send callable function requests to the cloudfunctions.net domain). If a function is doing something like setting a cookie on the client, the domain used when calling the function matters.

This PR looks like it addresses that need, but, if so, the name of the token FunctionsEmulatorOriginToken is inappropriate. Might I suggest renaming it to simply FunctionsOriginToken.

@jamesdaniels
Copy link
Copy Markdown
Member Author

jamesdaniels commented May 11, 2019

@thefliik I like that idea, I'm also not loving the *Token pattern. Might align more with Angular here and name it FUNCTIONS_ORIGIN rather than FunctionsOriginToken

@hiepxanh
Copy link
Copy Markdown
Contributor

hiepxanh commented May 13, 2019

i love this

@jamesdaniels jamesdaniels changed the title Allow configuration of Functions Emulator Origin feat(functions): Allow configuration of Functions Emulator Origin May 20, 2019
davideast
davideast approved these changes May 20, 2019
View reviewed changes
Copy link
Copy Markdown
Collaborator

@davideast davideast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jamesdaniels jamesdaniels merged commit d12b4c5 into master May 20, 2019
@jamesdaniels jamesdaniels deleted the functions_emulator branch May 20, 2019 20:47
@jamesdaniels jamesdaniels mentioned this pull request May 24, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@davideast davideast davideast approved these changes

Assignees

No one assigned

Labels

cla: yes

Projects

None yet

Milestone

5.2.0

Development

Successfully merging this pull request may close these issues.

6 participants

@jamesdaniels @Splaktar @jorroll @hiepxanh @davideast @googlebot