Skip to content

fix: Prevent invalid host assignment#9441

Merged
emrysal merged 12 commits intomainfrom
bugfix/prevent-invalid-hosts
Jun 10, 2023
Merged

fix: Prevent invalid host assignment#9441
emrysal merged 12 commits intomainfrom
bugfix/prevent-invalid-hosts

Conversation

@emrysal
Copy link
Copy Markdown
Contributor

@emrysal emrysal commented Jun 9, 2023

What does this PR do?

Adds extra checks while adding hosts.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 9, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2023 2:47am
cal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2023 2:47am
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2023 2:47am
web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2023 2:47am

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 9, 2023

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Jun 9, 2023

Current Playwright Test Results Summary

✅ 107 Passing - ⚠️ 2 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 06/10/2023 02:40:14am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: e89f7a9

Started: 06/10/2023 02:37:58am UTC

⚠️ Flakes

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should be able to reschedule
Retry 2Retry 1Initial Attempt
5.26% (10) 10 / 190 runs
failed over last 7 days
94.74% (180) 180 / 190 runs
flaked over last 7 days

📄   apps/web/playwright/login.2fa.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
2FA Tests should allow a user to disable 2FA
Retry 1Initial Attempt
2.07% (6) 6 / 290 runs
failed over last 7 days
22.76% (66) 66 / 290 runs
flaked over last 7 days

View Detailed Build Results


roae
roae previously requested changes Jun 10, 2023
Copy link
Copy Markdown
Contributor

@roae roae left a comment

Choose a reason for hiding this comment

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

this is failing because it is not getting prisma correctly, leave a comment in the line where the error is

Comment thread packages/trpc/server/routers/viewer/eventTypes/update.handler.ts Outdated
Copy link
Copy Markdown
Contributor

@roae roae left a comment

Choose a reason for hiding this comment

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

Nice job @emrysal 🙏

@emrysal emrysal added this pull request to the merge queue Jun 10, 2023
Merged via the queue into main with commit bb31c4b Jun 10, 2023
@emrysal emrysal deleted the bugfix/prevent-invalid-hosts branch June 10, 2023 02:51
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants