FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add support for App Check replay protection in callable functions by avolkovi · Pull Request #7296 · firebase/firebase-js-sdk · GitHub

/ firebase-js-sdk Public

Add support for App Check replay protection in callable functions - #7296

Merged
avolkovi merged 7 commits into
masterfrom
avolkovi-fac-functions
May 15, 2023
ConversationCommits (7)ChecksFiles changed
Merged

Add support for App Check replay protection in callable functions#7296
avolkovi merged 7 commits into
masterfrom
avolkovi-fac-functions

Conversation

avolkovi commented May 12, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

Hey there! So you want to contribute to a Firebase SDK?
Before you file this pull request, please read these guidelines:

Discussion

  • Read the contribution guidelines (CONTRIBUTING.md).
  • If this has been discussed in an issue, make sure to link to the issue here.
    If not, go file an issue about this before creating a pull request to discuss.

Testing

  • Make sure all existing tests in the repository pass after your change.
  • If you fixed a bug or added a feature, add a new test to cover your code.

API Changes

internal API proposal: go/fac-1-use-callable-sdk

avolkovi requested review from a team, dwyfrequency and hsubox76 as code owners May 12, 2023 16:47

changeset-bot Bot commented May 12, 2023 •
edited
Loading

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 2ca507c

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

This PR includes changesets to release 6 packages
Name Type
@firebase/app-check-interop-types Minor
@firebase/app-check Minor
@firebase/functions Minor
@firebase/app-check-compat 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

avolkovi force-pushed the avolkovi-fac-functions branch from b765a5d to f0119d6 Compare May 12, 2023 16:50
hsubox76 reviewed May 12, 2023
View reviewed changes

hsubox76 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Code LG, one suggestion to streamline the test.

For the PR process:

  • This will need a changeset (yarn changeset will automatically help you add the packages (should be patch for app-check and minor for functions) but you'll also need to add 'firebase': minor manually beneath those.
  • This will make a docs change, you'll need to run yarn docgen devsite and check in chagnes to the docs-devsite dir and have them reviewed by the appropriate tech writer for App Check which I think is @kevinthecheung

Comment thread packages/functions/src/callable.test.ts Outdated

google-oss-bot commented May 12, 2023 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Size Report 1

Affected Products

  • @firebase/app-check

    TypeBase (a9da1b7)Merge (84629f5)Diff
    browser26.2 kB26.2 kB+54 B (+0.2%)
    esm531.4 kB31.5 kB+69 B (+0.2%)
    main32.6 kB32.7 kB+69 B (+0.2%)
    module26.2 kB26.2 kB+54 B (+0.2%)

  • @firebase/functions

    TypeBase (a9da1b7)Merge (84629f5)Diff
    browser9.55 kB9.72 kB+170 B (+1.8%)
    esm511.8 kB12.1 kB+256 B (+2.2%)
    main12.6 kB12.9 kB+256 B (+2.0%)
    module9.55 kB9.72 kB+170 B (+1.8%)

  • bundle

    TypeBase (a9da1b7)Merge (84629f5)Diff
    app-check (CustomProvider)36.3 kB36.6 kB+254 B (+0.7%)
    app-check (ReCaptchaEnterpriseProvider)38.8 kB39.1 kB+254 B (+0.7%)
    app-check (ReCaptchaV3Provider)38.8 kB39.1 kB+254 B (+0.7%)
    functions (call)31.1 kB31.2 kB+72 B (+0.2%)

  • firebase

    TypeBase (a9da1b7)Merge (84629f5)Diff
    firebase-app-check-compat.js23.1 kB23.3 kB+246 B (+1.1%)
    firebase-app-check.js22.1 kB22.1 kB+34 B (+0.2%)
    firebase-compat.js773 kB773 kB+321 B (+0.0%)
    firebase-functions-compat.js8.12 kB8.19 kB+72 B (+0.9%)
    firebase-functions.js11.0 kB11.0 kB+72 B (+0.7%)

Test Logs

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

google-oss-bot commented May 12, 2023 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Size Analysis Report 1

Affected Products

  • @firebase/app-check

    • CustomProvider

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size7.51 kB7.65 kB+142 B (+1.9%)
      size-with-ext-deps25.2 kB25.4 kB+146 B (+0.6%)

      Dependency

      TypeBase (a9da1b7)Merge (84629f5)Diff
      functions

      20 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
factory
formatDummyToken
getDBPromise
getStateReference
getToken$2
initTokenRefresher
internalFactory
isValid
makeDummyTokenResult
notifyTokenListeners
registerAppCheck
removeTokenListener
sleep
write
writeTokenToIndexedDB
writeTokenToStorage

      21 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
factory
formatDummyToken
getDBPromise
getLimitedUseToken$1
getStateReference
getToken$2
initTokenRefresher
internalFactory
isValid
makeDummyTokenResult
notifyTokenListeners
registerAppCheck
removeTokenListener
sleep
write
writeTokenToIndexedDB
writeTokenToStorage

      + getLimitedUseToken$1

    • ReCaptchaEnterpriseProvider

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size11.2 kB11.3 kB+144 B (+1.3%)
      size-with-ext-deps28.8 kB28.9 kB+146 B (+0.5%)

      Dependency

      TypeBase (a9da1b7)Merge (84629f5)Diff
      functions

      33 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
exchangeToken
factory
formatDummyToken
getDBPromise
getDurationString
getExchangeRecaptchaEnterpriseTokenRequest
getRecaptcha
getStateReference
getToken$1
getToken$2
initTokenRefresher
initializeEnterprise
internalFactory
isValid
loadReCAPTCHAEnterpriseScript
makeDiv
makeDummyTokenResult
notifyTokenListeners
pad
queueWidgetRender
registerAppCheck
removeTokenListener
renderInvisibleWidget
setBackoff
sleep
throwIfThrottled
write
writeTokenToIndexedDB
writeTokenToStorage

      34 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
exchangeToken
factory
formatDummyToken
getDBPromise
getDurationString
getExchangeRecaptchaEnterpriseTokenRequest
getLimitedUseToken$1
getRecaptcha
getStateReference
getToken$1
getToken$2
initTokenRefresher
initializeEnterprise
internalFactory
isValid
loadReCAPTCHAEnterpriseScript
makeDiv
makeDummyTokenResult
notifyTokenListeners
pad
queueWidgetRender
registerAppCheck
removeTokenListener
renderInvisibleWidget
setBackoff
sleep
throwIfThrottled
write
writeTokenToIndexedDB
writeTokenToStorage

      + getLimitedUseToken$1

    • ReCaptchaV3Provider

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size11.2 kB11.3 kB+144 B (+1.3%)
      size-with-ext-deps28.7 kB28.9 kB+146 B (+0.5%)

      Dependency

      TypeBase (a9da1b7)Merge (84629f5)Diff
      functions

      33 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
exchangeToken
factory
formatDummyToken
getDBPromise
getDurationString
getExchangeRecaptchaV3TokenRequest
getRecaptcha
getStateReference
getToken$1
getToken$2
initTokenRefresher
initializeV3
internalFactory
isValid
loadReCAPTCHAV3Script
makeDiv
makeDummyTokenResult
notifyTokenListeners
pad
queueWidgetRender
registerAppCheck
removeTokenListener
renderInvisibleWidget
setBackoff
sleep
throwIfThrottled
write
writeTokenToIndexedDB
writeTokenToStorage

      34 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
exchangeToken
factory
formatDummyToken
getDBPromise
getDurationString
getExchangeRecaptchaV3TokenRequest
getLimitedUseToken$1
getRecaptcha
getStateReference
getToken$1
getToken$2
initTokenRefresher
initializeV3
internalFactory
isValid
loadReCAPTCHAV3Script
makeDiv
makeDummyTokenResult
notifyTokenListeners
pad
queueWidgetRender
registerAppCheck
removeTokenListener
renderInvisibleWidget
setBackoff
sleep
throwIfThrottled
write
writeTokenToIndexedDB
writeTokenToStorage

      + getLimitedUseToken$1

    • getLimitedUseToken

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size7.19 kB7.22 kB+28 B (+0.4%)
      size-with-ext-deps24.5 kB24.5 kB+29 B (+0.1%)

    • getToken

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size7.13 kB7.28 kB+142 B (+2.0%)
      size-with-ext-deps24.4 kB24.5 kB+146 B (+0.6%)

      Dependency

      TypeBase (a9da1b7)Merge (84629f5)Diff
      functions

      21 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
factory
formatDummyToken
getDBPromise
getStateReference
getToken
getToken$2
initTokenRefresher
internalFactory
isValid
makeDummyTokenResult
notifyTokenListeners
registerAppCheck
removeTokenListener
sleep
write
writeTokenToIndexedDB
writeTokenToStorage

      22 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
factory
formatDummyToken
getDBPromise
getLimitedUseToken$1
getStateReference
getToken
getToken$2
initTokenRefresher
internalFactory
isValid
makeDummyTokenResult
notifyTokenListeners
registerAppCheck
removeTokenListener
sleep
write
writeTokenToIndexedDB
writeTokenToStorage

      + getLimitedUseToken$1

    • initializeAppCheck

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size10.9 kB11.1 kB+249 B (+2.3%)
      size-with-ext-deps35.3 kB35.6 kB+254 B (+0.7%)

      Dependency

      TypeBase (a9da1b7)Merge (84629f5)Diff
      functions

      35 dependencies

      _activate
addTokenListener
computeKey
createTokenRefresher
ensureActivated
exchangeToken
factory
formatDummyToken
getDBPromise
getDebugState
getDebugToken
getExchangeDebugTokenRequest
getStateReference
getToken$2
initTokenRefresher
initializeAppCheck
initializeDebugMode
internalFactory
isDebugMode
isValid
makeDummyTokenResult
notifyTokenListeners
read
readDebugTokenFromIndexedDB
readOrCreateDebugTokenFromStorage
readTokenFromIndexedDB
readTokenFromStorage
registerAppCheck
removeTokenListener
setInitialState
sleep
write
writeDebugTokenToIndexedDB
writeTokenToIndexedDB
writeTokenToStorage

      36 dependencies

      _activate
addTokenListener
computeKey
createTokenRefresher
ensureActivated
exchangeToken
factory
formatDummyToken
getDBPromise
getDebugState
getDebugToken
getExchangeDebugTokenRequest
getLimitedUseToken$1
getStateReference
getToken$2
initTokenRefresher
initializeAppCheck
initializeDebugMode
internalFactory
isDebugMode
isValid
makeDummyTokenResult
notifyTokenListeners
read
readDebugTokenFromIndexedDB
readOrCreateDebugTokenFromStorage
readTokenFromIndexedDB
readTokenFromStorage
registerAppCheck
removeTokenListener
setInitialState
sleep
write
writeDebugTokenToIndexedDB
writeTokenToIndexedDB
writeTokenToStorage

      + getLimitedUseToken$1

    • onTokenChanged

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size7.23 kB7.37 kB+142 B (+2.0%)
      size-with-ext-deps24.5 kB24.6 kB+146 B (+0.6%)

      Dependency

      TypeBase (a9da1b7)Merge (84629f5)Diff
      functions

      21 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
factory
formatDummyToken
getDBPromise
getStateReference
getToken$2
initTokenRefresher
internalFactory
isValid
makeDummyTokenResult
notifyTokenListeners
onTokenChanged
registerAppCheck
removeTokenListener
sleep
write
writeTokenToIndexedDB
writeTokenToStorage

      22 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
factory
formatDummyToken
getDBPromise
getLimitedUseToken$1
getStateReference
getToken$2
initTokenRefresher
internalFactory
isValid
makeDummyTokenResult
notifyTokenListeners
onTokenChanged
registerAppCheck
removeTokenListener
sleep
write
writeTokenToIndexedDB
writeTokenToStorage

      + getLimitedUseToken$1

    • setTokenAutoRefreshEnabled

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size7.22 kB7.37 kB+142 B (+2.0%)
      size-with-ext-deps24.5 kB24.6 kB+146 B (+0.6%)

      Dependency

      TypeBase (a9da1b7)Merge (84629f5)Diff
      functions

      21 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
factory
formatDummyToken
getDBPromise
getStateReference
getToken$2
initTokenRefresher
internalFactory
isValid
makeDummyTokenResult
notifyTokenListeners
registerAppCheck
removeTokenListener
setTokenAutoRefreshEnabled
sleep
write
writeTokenToIndexedDB
writeTokenToStorage

      22 dependencies

      addTokenListener
computeKey
createTokenRefresher
ensureActivated
factory
formatDummyToken
getDBPromise
getLimitedUseToken$1
getStateReference
getToken$2
initTokenRefresher
internalFactory
isValid
makeDummyTokenResult
notifyTokenListeners
registerAppCheck
removeTokenListener
setTokenAutoRefreshEnabled
sleep
write
writeTokenToIndexedDB
writeTokenToStorage

      + getLimitedUseToken$1

  • @firebase/functions

    • connectFunctionsEmulator

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size2.42 kB2.46 kB+46 B (+1.9%)
      size-with-ext-deps19.5 kB19.5 kB+46 B (+0.2%)

    • getFunctions

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size2.60 kB2.65 kB+46 B (+1.8%)
      size-with-ext-deps27.3 kB27.3 kB+46 B (+0.2%)

    • httpsCallable

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size6.02 kB6.09 kB+72 B (+1.2%)
      size-with-ext-deps23.1 kB23.2 kB+72 B (+0.3%)

    • httpsCallableFromURL

      Size

      TypeBase (a9da1b7)Merge (84629f5)Diff
      size5.97 kB6.04 kB+72 B (+1.2%)
      size-with-ext-deps23.0 kB23.1 kB+72 B (+0.3%)

Test Logs

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

avolkovi requested a review from hsubox76 May 12, 2023 17:49
hsubox76 approved these changes May 15, 2023
View reviewed changes

hsubox76 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LG - make sure you get tech writer review before merging.
Also can you add a description? Can just be a link to the design doc/API proposal, it's fine if it's internal, you can just say (internal doc)

kevinthecheung approved these changes May 15, 2023
View reviewed changes
Comment thread packages/functions/src/public-types.ts
Doc feedback
178e420 
Co-authored-by: Kevin Cheung <kevinthecheung@users.noreply.github.com>
avolkovi force-pushed the avolkovi-fac-functions branch from fb6e91b to 178e420 Compare May 15, 2023 22:11
weixifan reviewed May 15, 2023
View reviewed changes
Comment thread common/api-review/functions.api.md Outdated
Comment thread docs-devsite/functions.httpscallableoptions.md Outdated
weixifan approved these changes May 15, 2023
View reviewed changes
Comment thread packages/app-check-interop-types/index.d.ts Outdated
avolkovi merged commit e12e7f5 into master May 15, 2023
avolkovi deleted the avolkovi-fac-functions branch May 15, 2023 23:25
google-oss-bot mentioned this pull request May 23, 2023
Merged
firebase locked and limited conversation to collaborators Jun 15, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

hsubox76 hsubox76 approved these changes
kevinthecheung kevinthecheung approved these changes
weixifan weixifan approved these changes
dwyfrequency Awaiting requested review from dwyfrequency
egilmorez Awaiting requested review from egilmorez
markarndt Awaiting requested review from markarndt

Assignees

avolkovi

hsubox76

kevinthecheung

ssbushi

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

Back | FazBrowse Home | New Git URL