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

feat: migrate to hcaptcha-loader #144

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

CAMOBAP
Copy link
Collaborator

@CAMOBAP CAMOBAP commented Jan 26, 2024

No description provided.

@CAMOBAP CAMOBAP self-assigned this Jan 26, 2024
Copy link

github-actions bot commented Jan 26, 2024

Diffuse report:

OLD: sdk-main.aar
NEW: sdk-pr.aar

 AAR      │ old       │ new       │ diff       
──────────┼───────────┼───────────┼────────────
      jar │  51.8 KiB │  52.6 KiB │     +767 B 
 manifest │     411 B │     411 B │        0 B 
      res │  78.3 KiB │  78.3 KiB │        0 B 
    other │     930 B │     930 B │        0 B 
──────────┼───────────┼───────────┼────────────
    total │ 131.4 KiB │ 301.5 KiB │ +170.1 KiB 

 JAR     │ old │ new │ diff         
─────────┼─────┼─────┼──────────────
 classes │  37 │  37 │   0 (+0 -0)  
 methods │ 507 │ 517 │ +10 (+11 -1) 
  fields │ 163 │ 165 │  +2 (+2 -0)
AAR
 size      │ diff       │ path                           
───────────┼────────────┼────────────────────────────────
 126.2 KiB │ +126.2 KiB │ + assets/hcaptcha/loader.js    
  43.2 KiB │  +43.2 KiB │ + assets/hcaptcha/polyfills.js 
       0 B │        0 B │ + assets/hcaptcha/             
       0 B │        0 B │ + assets/                      
  52.6 KiB │     +767 B │ ∆ classes.jar                  
───────────┼────────────┼────────────────────────────────
   222 KiB │ +170.1 KiB │ (total)
JAR
METHODS:

   old │ new │ diff         
  ─────┼─────┼──────────────
   507 │ 517 │ +10 (+11 -1) 
  + com.hcaptcha.sdk.HCaptchaWebViewHelper access_100(HCaptchaWebViewHelper) → HCaptchaConfig
  + com.hcaptcha.sdk.HCaptchaWebViewHelper access_200(HCaptchaWebViewHelper) → HCaptchaWebView
  + android.content.Context getAssets() → AssetManager
  + android.content.res.AssetManager open(String) → InputStream
  + android.net.Uri parse(String) → Uri
  + android.view.View getContext() → Context
  + android.webkit.WebResourceResponse <init>(String, String, int, String, Map, InputStream)
  + java.util.Collections singletonMap(Object, Object) → Map
  + java.util.Map get(Object) → Object
  + java.util.Map put(Object, Object) → Object
  + java.util.Objects toString(Object, String) → String
  
  - com.hcaptcha.sdk.HCaptchaWebViewHelper access_100(HCaptchaWebViewHelper) → HCaptchaWebView
  

FIELDS:

   old │ new │ diff       
  ─────┼─────┼────────────
   163 │ 165 │ +2 (+2 -0) 
  + com.hcaptcha.sdk.BuildConfig LOADER_VERSION: String
  + com.hcaptcha.sdk.HCaptchaWebViewHelper_HCaptchaWebClient assetsCache: Map

Copy link

github-actions bot commented Jan 26, 2024

Benchmark report:

Test name Time ms. (median) Allocations (median)
com.hcaptcha.sdk.HCaptchaBenchmarkTest.EMULATOR_UNLOCKED_benchmarkInvisibleVerification +547.80 -6
com.hcaptcha.sdk.HCaptchaBenchmarkTest.EMULATOR_UNLOCKED_benchmarkInvisibleVerificationColdRun +391.46 +107
com.hcaptcha.sdk.HCaptchaBenchmarkTest.EMULATOR_UNLOCKED_benchmarkInvisibleSetup +2.81 +102.25
com.hcaptcha.sdk.HCaptchaDebugInfoTest.EMULATOR_UNLOCKED_benchmarkDebugInfo -6.30 +556
com.hcaptcha.sdk.HCaptchaDebugInfoTest.EMULATOR_UNLOCKED_benchmarkDebugSys +33.41 +3
com.hcaptcha.sdk.HCaptchaWebViewHelperTest.EMULATOR_UNLOCKED_benchmarkWebViewLoad +812.08 +103

@e271828-
Copy link
Contributor

e271828- commented Jan 26, 2024

Waiting for next release of loader with less bloated sentry.

Sanalbahis06

This comment was marked as spam.

@e271828- e271828- marked this pull request as draft January 29, 2024 13:24
@CAMOBAP CAMOBAP requested a review from DSergiu February 29, 2024 21:30
@CAMOBAP CAMOBAP marked this pull request as ready for review February 29, 2024 21:31
Copy link

sonarcloud bot commented Feb 29, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants