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

SSL error: The certificate authority is not trusted #2012

Closed
coomac opened this issue Nov 24, 2020 · 10 comments
Closed

SSL error: The certificate authority is not trusted #2012

coomac opened this issue Nov 24, 2020 · 10 comments
Labels
community Issues or PRs opened by the MM community stale Issues that have not had activity in the last 90 days type-bug Something isn't working

Comments

@coomac
Copy link

coomac commented Nov 24, 2020

Describe the bug
This error message appears when I try to use the built in browser on metamask android to visit any website:

Oops! something went wrong...
We weren't able to load that page.
Reason: SSL error: The certificate authority is not trusted.

Screenshot
Screenshot_20201124-140444_Pixel_Launcher

To Reproduce

  1. Open metamask
  2. Click on menu and select Browser
  3. Page loads for a second and immediately refreshes to show the error.
    The error appears on the homepage (https://home.metamask.io) as well as on any other website manually typed into the address bar. No websites can be accessed because the error cannot be bypassed.

Expected behavior
Website should load without an error.

Smartphone (please complete the following information):

  • Device: OnePlus 6
  • OS: Android 11, custom rom (Evolution) with Nov 5 Android security update
  • App Version - v1.0.7 (40)

Not sure if this is a problem introduced with the Nov 19 metamask update or from the rom (both were installed around the same time). I have noticed other wallets that are somewhat incompatible with various roms running the Nov security update.


to be added after bug submission by internal support / PM
Severity

  • How critical is the impact of this bug on a user?
  • Add stats if available on % of customers impacted
  • Is this visible to all users?
  • Is this tech debt?
@coomac coomac added the type-bug Something isn't working label Nov 24, 2020
@omnat
Copy link
Contributor

omnat commented Nov 25, 2020

Hi @coomac thanks for reporting. Could you try closing all tabs you might have open in your MetaMask browser, quit the app, and try again?
Which sites have you tried accessing that are giving this error?

@coomac
Copy link
Author

coomac commented Nov 25, 2020 via email

@gantunesr gantunesr added the community Issues or PRs opened by the MM community label Sep 14, 2021
@sharpshark02020
Copy link

Is there any solutiom to this? I have this same issue on metamask app on different devices these 2 days and I didn't root any of my devices. Thamks

@coomac
Copy link
Author

coomac commented Oct 7, 2021

Is there any solutiom to this? I have this same issue on metamask app on different devices these 2 days and I didn't root any of my devices. Thamks

Are you using a pihole or some other system-wide adblocker? In my case, my adblocker was blocking a url related to the certificate (only made the connection because a somewhat similar issue popped up when logging in on another app). I just switched block lists.

@sharpshark02020
Copy link

Make sense. However, I never installed such app in my devices.
1 thing I noticed is, the problem occurs at the same time on both the devices. Some updates might push to metamask app, not sure.

What do you mean by switching block lists, and where did you do that?

@coomac
Copy link
Author

coomac commented Oct 12, 2021

I was using Adaway at the time and had to disable one of the adblock lists in the app. Don't recall which list since I now use Next DNS.

Does the error show up randomly but the websites work at other times? Also, do you get the same error on both devices at the same time regardless of if they're connected to data, your home wifi, work or some random cafe wifi?

@derek2035
Copy link

I'm debugging my local DAPP. I use the HTTPS certificate generated locally by mkcert, which also reports this error. Is there a way to solve it, thank you

@lxjwlt
Copy link

lxjwlt commented Aug 15, 2022

I have tried to modify some code underneath and recompiled, it works fine finally:

  <base-config cleartextTrafficPermitted="false">
    <trust-anchors>
      <certificates src="system" />
      <certificates src="user" />
    </trust-anchors>
  </base-config>

image

you guys could download this recompiled apk from here, only for development.

@royaldragonit
Copy link

Hey everyone.
I'm make sure 100% this error only show in android device.
To the fix problem, you need re-configure ssl for your domain.
Can general at here .
I tried and working.

@sethkfman sethkfman added the stale Issues that have not had activity in the last 90 days label Jun 9, 2023
@github-actions
Copy link
Contributor

This issue was closed because it has been stalled for 7 days with no activity. If you feel this was closed in error please reopen and provide evidence on the current production app. Thank you for your contributions.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jun 17, 2023
github-merge-queue bot pushed a commit that referenced this issue Nov 4, 2024
## **Description**

Display staking transaction methods in the transaction confirmation
rather than `Unknown Method`.

Specifically:

- Provide new `TransactionType` values in the `addTransaction` options.
- Add action keys for staking transaction types.
- Upgrade `@metamask/transaction-controller` to `38.2.0`.

## **Related issues**

Fixes: [#2012](MetaMask/mobile-planning#2012)

## **Manual testing steps**

1. Add `export MM_POOLED_STAKING_UI_ENABLED="true"` to `.js.env`.
2. Create each staking transaction type.
3. Verify method is correct and not `Unknown Method`.

## **Screenshots/Recordings**

### **Before**

<img width="200" alt="Before"
src="https://github.com/user-attachments/assets/c68c7f06-eb73-4d09-b02e-939ac4232a1c">

### **After**

<img width="200" alt="Deposit"
src="https://github.com/user-attachments/assets/3cf566eb-8269-4ea9-a2bf-246cbf80f787">

<img width="200" alt="Claim"
src="https://github.com/user-attachments/assets/5704717d-e037-4e25-9ad2-b345fd53d8cf">

<img width="200" alt="Unstake"
src="https://github.com/user-attachments/assets/5d0f0e5e-138f-4913-93b6-9fc28233b244">

<img width="200" alt="Activity List"
src="https://github.com/user-attachments/assets/6d8c7224-4fdb-45c2-8b30-2cdeae0eb97c">

## **Pre-merge author checklist**

- [x] I’ve followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.
matthewwalsh0 added a commit that referenced this issue Nov 5, 2024
## **Description**

Display staking transaction methods in the transaction confirmation
rather than `Unknown Method`.

Specifically:

- Provide new `TransactionType` values in the `addTransaction` options.
- Add action keys for staking transaction types.
- Upgrade `@metamask/transaction-controller` to `38.2.0`.

## **Related issues**

Fixes: [#2012](MetaMask/mobile-planning#2012)

## **Manual testing steps**

1. Add `export MM_POOLED_STAKING_UI_ENABLED="true"` to `.js.env`.
2. Create each staking transaction type.
3. Verify method is correct and not `Unknown Method`.

## **Screenshots/Recordings**

### **Before**

<img width="200" alt="Before"
src="https://github.com/user-attachments/assets/c68c7f06-eb73-4d09-b02e-939ac4232a1c">

### **After**

<img width="200" alt="Deposit"
src="https://github.com/user-attachments/assets/3cf566eb-8269-4ea9-a2bf-246cbf80f787">

<img width="200" alt="Claim"
src="https://github.com/user-attachments/assets/5704717d-e037-4e25-9ad2-b345fd53d8cf">

<img width="200" alt="Unstake"
src="https://github.com/user-attachments/assets/5d0f0e5e-138f-4913-93b6-9fc28233b244">

<img width="200" alt="Activity List"
src="https://github.com/user-attachments/assets/6d8c7224-4fdb-45c2-8b30-2cdeae0eb97c">

## **Pre-merge author checklist**

- [x] I’ve followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding
Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Issues or PRs opened by the MM community stale Issues that have not had activity in the last 90 days type-bug Something isn't working
Projects
None yet
Development

No branches or pull requests

8 participants