Skip to content
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

Add deprecation tagging to SafetyNetAppCheckProviderFactory. #4686

Merged
merged 2 commits into from
Feb 16, 2023

Conversation

rosalyntan
Copy link
Member

Per discussion, add deprecation tags to the App Check SafetyNet SDK.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 15, 2023

Javadoc Changes:
--- /home/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html	2023-02-15 21:56:24.046218518 +0000
+++ /home/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html	2023-02-15 21:54:43.734830283 +0000
@@ -10,6 +10,9 @@
       <pre>public class SafetyNetAppCheckProviderFactory implements <a href="/docs/reference/android/com/google/firebase/appcheck/AppCheckProviderFactory.html">AppCheckProviderFactory</a></pre>
     </p>
     <hr>
+    <aside class="caution"><strong>This class is deprecated.</strong><br>
+      <p>Use <code>PlayIntegrityAppCheckProviderFactory</code> instead.</p>
+    </aside>
     <p>Implementation of an <code><a href="/docs/reference/android/com/google/firebase/appcheck/AppCheckProviderFactory.html">AppCheckProviderFactory</a></code> that builds SafetyNetAppCheckProviders. This is the default implementation.</p>
     <h2>Summary</h2>
     <div class="devsite-table-wrapper">
@@ -39,14 +42,16 @@
           <tr>
             <td width="40%"><code>@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/appcheck/AppCheckProvider.html">AppCheckProvider</a></code></td>
             <td>
-              <div><code><a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#create(com.google.firebase.FirebaseApp)">create</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a>&nbsp;firebaseApp)</code></div>
+              <div><code><span><del><a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#create(com.google.firebase.FirebaseApp)">create</a></del></span>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a>&nbsp;firebaseApp)</code></div>
             </td>
           </tr>
           <tr>
             <td width="40%"><code>static @<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html">SafetyNetAppCheckProviderFactory</a></code></td>
             <td>
-              <div><code><a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#getInstance()">getInstance</a>()</code></div>
-              <p>Gets an instance of this class for installation into a  instance.</p>
+              <div><code><span><del><a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#getInstance()">getInstance</a></del></span>()</code></div>
+              <p><strong>This method is deprecated.</strong>
+                <p>Use <code>PlayIntegrityAppCheckProviderFactory#getInstance</code> instead.</p>
+              </p>
             </td>
           </tr>
         </tbody>
@@ -60,11 +65,14 @@
     <h2>Public methods</h2>
     <div><a name="create-com.google.firebase.FirebaseApp-"></a><a name="create"></a>
       <h3 class="api-name" id="create(com.google.firebase.FirebaseApp)">create</h3>
-      <pre class="api-signature no-pretty-print">public&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/appcheck/AppCheckProvider.html">AppCheckProvider</a>&nbsp;<a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#create(com.google.firebase.FirebaseApp)">create</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a>&nbsp;firebaseApp)</pre>
+      <pre class="api-signature no-pretty-print">public&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/appcheck/AppCheckProvider.html">AppCheckProvider</a>&nbsp;<span><del><a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#create(com.google.firebase.FirebaseApp)">create</a></del></span>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/FirebaseApp.html">FirebaseApp</a>&nbsp;firebaseApp)</pre>
     </div>
     <div><a name="getInstance--"></a><a name="getinstance"></a>
       <h3 class="api-name" id="getInstance()">getInstance</h3>
-      <pre class="api-signature no-pretty-print">public&nbsp;static&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html">SafetyNetAppCheckProviderFactory</a>&nbsp;<a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#getInstance()">getInstance</a>()</pre>
+      <pre class="api-signature no-pretty-print">public&nbsp;static&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html">SafetyNetAppCheckProviderFactory</a>&nbsp;<span><del><a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#getInstance()">getInstance</a></del></span>()</pre>
+      <aside class="caution"><strong>This method is deprecated.</strong><br>
+        <p>Use <code>PlayIntegrityAppCheckProviderFactory#getInstance</code> instead.</p>
+      </aside>
       <p>Gets an instance of this class for installation into a  instance.</p>
     </div>
   </body>
--- /home/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/appcheck/safetynet/package-summary.html	2023-02-15 21:56:24.046218518 +0000
+++ /home/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/appcheck/safetynet/package-summary.html	2023-02-15 21:54:43.738830259 +0000
@@ -13,7 +13,9 @@
           <tr>
             <td width="40%"><code><a href="/docs/reference/android/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html">SafetyNetAppCheckProviderFactory</a></code></td>
             <td>
-              <p>Implementation of an <code><a href="/docs/reference/android/com/google/firebase/appcheck/AppCheckProviderFactory.html">AppCheckProviderFactory</a></code> that builds SafetyNetAppCheckProviders.</p>
+              <p><strong>This class is deprecated.</strong>
+                <p>Use <code>PlayIntegrityAppCheckProviderFactory</code> instead.</p>
+              </p>
             </td>
           </tr>
         </tbody>
--- /home/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html	2023-02-15 21:56:24.066218397 +0000
+++ /home/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html	2023-02-15 21:54:43.770830067 +0000
@@ -10,6 +10,9 @@
       <pre>class SafetyNetAppCheckProviderFactory : <a href="/docs/reference/kotlin/com/google/firebase/appcheck/AppCheckProviderFactory.html">AppCheckProviderFactory</a></pre>
     </p>
     <hr>
+    <aside class="caution"><strong>This class is deprecated.</strong><br>
+      <p>Use <code>PlayIntegrityAppCheckProviderFactory</code> instead.</p>
+    </aside>
     <p>Implementation of an <code><a href="/docs/reference/kotlin/com/google/firebase/appcheck/AppCheckProviderFactory.html">AppCheckProviderFactory</a></code> that builds SafetyNetAppCheckProviders. This is the default implementation.</p>
     <h2>Summary</h2>
     <div class="devsite-table-wrapper">
@@ -39,14 +42,16 @@
           <tr>
             <td width="40%"><code><a href="/docs/reference/kotlin/com/google/firebase/appcheck/AppCheckProvider.html">AppCheckProvider</a></code></td>
             <td>
-              <div><code><a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#create(com.google.firebase.FirebaseApp)">create</a>(firebaseApp:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a>)</code></div>
+              <div><code><span><del><a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#create(com.google.firebase.FirebaseApp)">create</a></del></span>(firebaseApp:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a>)</code></div>
             </td>
           </tr>
           <tr>
             <td width="40%"><code>java-static <a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html">SafetyNetAppCheckProviderFactory</a></code></td>
             <td>
-              <div><code><a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#getInstance()">getInstance</a>()</code></div>
-              <p>Gets an instance of this class for installation into a  instance.</p>
+              <div><code><span><del><a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#getInstance()">getInstance</a></del></span>()</code></div>
+              <p><strong>This function is deprecated.</strong>
+                <p>Use <code>PlayIntegrityAppCheckProviderFactory#getInstance</code> instead.</p>
+              </p>
             </td>
           </tr>
         </tbody>
@@ -60,11 +65,14 @@
     <h2>Public functions</h2>
     <div><a name="create-com.google.firebase.FirebaseApp-"></a><a name="create"></a>
       <h3 class="api-name" id="create(com.google.firebase.FirebaseApp)">create</h3>
-      <pre class="api-signature no-pretty-print">fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#create(com.google.firebase.FirebaseApp)">create</a>(firebaseApp:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a>):&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/appcheck/AppCheckProvider.html">AppCheckProvider</a></pre>
+      <pre class="api-signature no-pretty-print">fun&nbsp;<span><del><a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#create(com.google.firebase.FirebaseApp)">create</a></del></span>(firebaseApp:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/FirebaseApp.html">FirebaseApp</a>):&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/appcheck/AppCheckProvider.html">AppCheckProvider</a></pre>
     </div>
     <div><a name="getInstance--"></a><a name="getinstance"></a>
       <h3 class="api-name" id="getInstance()">getInstance</h3>
-      <pre class="api-signature no-pretty-print">java-static&nbsp;fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#getInstance()">getInstance</a>():&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html">SafetyNetAppCheckProviderFactory</a></pre>
+      <pre class="api-signature no-pretty-print">java-static&nbsp;fun&nbsp;<span><del><a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html#getInstance()">getInstance</a></del></span>():&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html">SafetyNetAppCheckProviderFactory</a></pre>
+      <aside class="caution"><strong>This function is deprecated.</strong><br>
+        <p>Use <code>PlayIntegrityAppCheckProviderFactory#getInstance</code> instead.</p>
+      </aside>
       <p>Gets an instance of this class for installation into a  instance.</p>
     </div>
   </body>
--- /home/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/appcheck/safetynet/package-summary.html	2023-02-15 21:56:24.066218397 +0000
+++ /home/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/appcheck/safetynet/package-summary.html	2023-02-15 21:54:43.770830067 +0000
@@ -13,7 +13,9 @@
           <tr>
             <td width="40%"><code><a href="/docs/reference/kotlin/com/google/firebase/appcheck/safetynet/SafetyNetAppCheckProviderFactory.html">SafetyNetAppCheckProviderFactory</a></code></td>
             <td>
-              <p>Implementation of an <code><a href="/docs/reference/kotlin/com/google/firebase/appcheck/AppCheckProviderFactory.html">AppCheckProviderFactory</a></code> that builds SafetyNetAppCheckProviders.</p>
+              <p><strong>This class is deprecated.</strong>
+                <p>Use <code>PlayIntegrityAppCheckProviderFactory</code> instead.</p>
+              </p>
             </td>
           </tr>
         </tbody>

@github-actions
Copy link
Contributor

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

Add the 'main-merge-ack' label to your PR to confirm merging into the main branch is intended.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Feb 15, 2023

Coverage Report 1

Affected Products

  • firebase-appcheck

    Overall coverage changed from ? (6071e9d) to 82.08% (7064085) by ?.

    18 individual files with coverage change

    FilenameBase (6071e9d)Merge (7064085)Diff
    AppCheckProvider.java?0.00%?
    AppCheckProviderFactory.java?0.00%?
    AppCheckToken.java?100.00%?
    AppCheckTokenResponse.java?100.00%?
    Clock.java?100.00%?
    DefaultAppCheckToken.java?87.76%?
    DefaultAppCheckTokenResult.java?100.00%?
    DefaultFirebaseAppCheck.java?82.73%?
    DefaultTokenRefresher.java?96.55%?
    FirebaseAppCheck.java?100.00%?
    FirebaseAppCheckRegistrar.java?100.00%?
    HttpErrorResponse.java?100.00%?
    Logger.java?32.43%?
    NetworkClient.java?77.78%?
    RetryManager.java?88.00%?
    StorageHelper.java?92.86%?
    TokenParser.java?73.91%?
    TokenRefreshManager.java?80.95%?

  • firebase-appcheck-debug

    Overall coverage changed from ? (6071e9d) to 75.31% (7064085) by ?.

    FilenameBase (6071e9d)Merge (7064085)Diff
    DebugAppCheckProvider.java?69.05%?
    DebugAppCheckProviderFactory.java?66.67%?
    ExchangeDebugTokenRequest.java?100.00%?
    FirebaseAppCheckDebugRegistrar.java?71.43%?
    InternalDebugSecretProvider.java?0.00%?
    StorageHelper.java?100.00%?
  • firebase-appcheck-debug-testing

    Overall coverage changed from ? (6071e9d) to 88.46% (7064085) by ?.

    FilenameBase (6071e9d)Merge (7064085)Diff
    DebugAppCheckTestHelper.java?82.35%?
    DebugSecretProvider.java?100.00%?
    FirebaseAppCheckDebugTestingRegistrar.java?100.00%?
  • firebase-appcheck-playintegrity

    Overall coverage changed from ? (6071e9d) to 89.47% (7064085) by ?.

    FilenameBase (6071e9d)Merge (7064085)Diff
    ExchangePlayIntegrityTokenRequest.java?100.00%?
    FirebaseAppCheckPlayIntegrityRegistrar.java?75.00%?
    GeneratePlayIntegrityChallengeRequest.java?100.00%?
    GeneratePlayIntegrityChallengeResponse.java?100.00%?
    PlayIntegrityAppCheckProvider.java?90.91%?
    PlayIntegrityAppCheckProviderFactory.java?75.00%?
  • firebase-appcheck-safetynet

    Overall coverage changed from ? (6071e9d) to 81.93% (7064085) by ?.

    FilenameBase (6071e9d)Merge (7064085)Diff
    ExchangeSafetyNetTokenRequest.java?100.00%?
    FirebaseAppCheckSafetyNetRegistrar.java?73.68%?
    SafetyNetAppCheckProvider.java?83.64%?
    SafetyNetAppCheckProviderFactory.java?66.67%?

Test Logs

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

@github-actions
Copy link
Contributor

github-actions bot commented Feb 15, 2023

Unit Test Results

  44 files   -    311    44 suites   - 311   4m 10s ⏱️ - 13m 26s
237 tests  - 4 189  237 ✔️  - 4 168  0 💤  - 21  0 ±0 
237 runs   - 4 205  237 ✔️  - 4 184  0 💤  - 21  0 ±0 

Results for commit 286a332. ± Comparison against base commit 7d98d5f.

♻️ This comment has been updated with latest results.

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • firebase-appcheck-safetynet

    TypeBase (6071e9d)Merge (7064085)Diff
    aar10.4 kB10.4 kB+42 B (+0.4%)
    apk (release)1.57 MB1.57 MB+16 B (+0.0%)

Test Logs

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

@google-oss-bot
Copy link
Contributor

Startup Time Report 1

Note: Layout is sometimes suboptimal due to limited formatting support on GitHub. Please check this report on GCS.

Startup time comparison between the CI merge commit (7064085) and the base commit (6071e9d) are not available.

No macrobenchmark data found for the base commit (6071e9d). Analysis for the CI merge commit (7064085) can be found at:

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

@rosalyntan rosalyntan merged commit 7e70d57 into master Feb 16, 2023
@rosalyntan rosalyntan deleted the rosalyntan.safetynet branch February 16, 2023 00:01
@firebase firebase locked and limited conversation to collaborators Mar 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants