Skip to content

Helix OSX - Playwright ERR_CERT_DATABASE_CHANGED failures Individual Auth #30825

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

Closed
HaoK opened this issue Mar 11, 2021 · 9 comments · Fixed by #31012
Closed

Helix OSX - Playwright ERR_CERT_DATABASE_CHANGED failures Individual Auth #30825

HaoK opened this issue Mar 11, 2021 · 9 comments · Fixed by #31012
Labels
area-blazor Includes: Blazor, Razor Components feature-playwright 🎭 This issue is related to Playwright E2E Project Template Tests ✔️ Resolution: Duplicate Resolved as a duplicate of another issue Pillar: Technical Debt Priority:0 Work that we can't release without Status: Resolved test-failure
Milestone

Comments

@HaoK
Copy link
Member

HaoK commented Mar 11, 2021

IndividualAuth_Works tests are failing with the following on OSX, this seems like its generally an issue when the trust store is modified which causes Chrome to flush all the connections, we should investigate to see if we can avoid these exceptions

PlaywrightSharp.PlaywrightSharpException : net::ERR_CERT_DATABASE_CHANGED at http://localhost:63342/
=========================== logs ===========================
navigating to "http://localhost:63342/", waiting until "load"

  at PlaywrightSharp.Transport.Connection.SendMessageToServerAsync[T](String guid, String method, Object args, Boolean ignoreNullValues, JsonSerializerOptions serializerOptions, Boolean treatErrorPropertyAsError) in /home/runner/work/playwright-sharp/playwright-sharp/src/PlaywrightSharp/Transport/Connection.cs:line 186
   at PlaywrightSharp.Frame.GoToAsync(Boolean isPage, String url, Nullable`1 waitUntil, String referer, Nullable`1 timeout) in /home/runner/work/playwright-sharp/playwright-sharp/src/PlaywrightSharp/Frame.cs:line 761
   at Templates.Test.Helpers.AspNetProcess.VisitInBrowserAsync(IPage page) in /_/src/ProjectTemplates/Shared/AspNetProcess.cs:line 108
   at Templates.Test.BlazorServerTemplateTest.BlazorServerTemplateWorks_IndividualAuth(BrowserKind browserKind, Boolean useLocalDB) in /_/src/ProjectTemplates/BlazorTemplates.Tests/BlazorServerTemplateTest.cs:line 164
--- End of stack trace from previous location ---
@javiercn
Copy link
Member

I think I ran into this in my prototype, but there's an easy solution for it.

We create a cert every time we spin up a .NET project, we can switch to create a single https certificate and use that for all of our tests instead.

@mkArtakMSFT mkArtakMSFT added this to the Next sprint planning milestone Mar 11, 2021
@ghost
Copy link

ghost commented Mar 11, 2021

Thanks for contacting us.
We're moving this issue to the Next sprint planning milestone for future evaluation / consideration. We will evaluate the request when we are planning the work for the next milestone. To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@HaoK HaoK self-assigned this Mar 18, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Apr 18, 2021
@TanayParikh
Copy link
Contributor

Re-opening this issue as it's still used to mark tests as quarantined.

  • BlazorServerTemplateWorks_IndividualAuth
  • BlazorWasmHostedTemplate_IndividualAuth_Works_WithOutLocalDB
  • BlazorServerTemplateWorks_IndividualAuth

@SteveSandersonMS
Copy link
Member

@TanayParikh I see this has the test-fixed label but could you clarify which PR contained the fix? I'm unclear on how to know if it's been 30 days since this was fixed.

@TanayParikh
Copy link
Contributor

TanayParikh commented Nov 18, 2021

which PR contained the fix

#31012

I'm unclear on how to know if it's been 30 days since this was fixed.

This issue was closed out when the above PR was merged, but the underlying tests were never un-quarantined. Given it was merged in March, assuming all the tests have a consistent pass rate, we should be fine to un-quarantine now.

@mkArtakMSFT mkArtakMSFT modified the milestones: .NET 7 Planning, 7.0-rc2 Aug 10, 2022
@mkArtakMSFT mkArtakMSFT modified the milestones: 7.0-rc2, .NET 8 Planning Sep 7, 2022
@mkArtakMSFT mkArtakMSFT modified the milestones: .NET 8 Planning, 8.0-MQ Sep 15, 2022
@TanayParikh TanayParikh closed this as not planned Won't fix, can't repro, duplicate, stale Dec 21, 2022
@TanayParikh
Copy link
Contributor

We need Helix enabled for the Playwright tests first before we resolve this issue (if this is still an issue):

Depends on: #38818

@TanayParikh TanayParikh reopened this Dec 21, 2022
@mkArtakMSFT mkArtakMSFT modified the milestones: 8.0-MQ, 8.0-preview1 Jan 4, 2023
@TanayParikh TanayParikh modified the milestones: 8.0-preview2, Backlog Feb 1, 2023
@TanayParikh TanayParikh removed their assignment Feb 1, 2023
@TanayParikh
Copy link
Contributor

We need Helix enabled for the Playwright tests first before we resolve this issue (if this is still an issue):

We're running the tests specifically on the build agents for now due to issues with Helix compatibility. Backlogging this until we decide we'd like these tests running on helix.

@javiercn
Copy link
Member

javiercn commented Feb 6, 2023

For reference here: I believe the issue is that we generate a new HTTPS cert for each test and that causes issues on chrome. We can generate a single HTTPS cert at the beginning of the test run and that should get rid of these problems.

@mkArtakMSFT mkArtakMSFT modified the milestones: Backlog, BlazorPlanning Nov 5, 2023
@mkArtakMSFT mkArtakMSFT added the Priority:0 Work that we can't release without label Jan 3, 2024
@mkArtakMSFT
Copy link
Contributor

Closing as a dupe of #46430

@mkArtakMSFT mkArtakMSFT closed this as not planned Won't fix, can't repro, duplicate, stale Feb 20, 2024
@mkArtakMSFT mkArtakMSFT added the ✔️ Resolution: Duplicate Resolved as a duplicate of another issue label Feb 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-blazor Includes: Blazor, Razor Components feature-playwright 🎭 This issue is related to Playwright E2E Project Template Tests ✔️ Resolution: Duplicate Resolved as a duplicate of another issue Pillar: Technical Debt Priority:0 Work that we can't release without Status: Resolved test-failure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants