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.

Sign up for GitHub

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

Jump to bottom

Duplicated frontmatter field causes the dev server to crash #6587

Closed
1 task
MoustaphaDev opened this issue Mar 17, 2023 · 2 comments · Fixed by #6675
Closed
1 task

Duplicated frontmatter field causes the dev server to crash #6587

MoustaphaDev opened this issue Mar 17, 2023 · 2 comments · Fixed by #6675
Assignees
@matthewp
Labels
- P3: minor bug An edge case that only affects very specific usage (priority)

Comments

@MoustaphaDev
Copy link
Member

What version of astro are you using?

2.1.3

Are you using an SSR adapter? If so, which one?

None

What package manager are you using?

pnpm

What operating system are you using?

Ubuntu 22.04

What browser are you using?

Irrelevant

Describe the Bug

Having duplicated frontmatter keys in a markdown or mdx file causes the dev server to crash.

Link to Minimal Reproducible Example

https://stackblitz.com/edit/github-pbwi2s?file=src%2Fcontent%2Fblog%2Ffirst-post.md,src%2Fpages%2Findex.astro,package.json&on=stackblitz

Participation

  • I am willing to submit a pull request for this issue.
@MoustaphaDev MoustaphaDev added the - P3: minor bug An edge case that only affects very specific usage (priority) label Mar 17, 2023
@MoustaphaDev MoustaphaDev changed the title Duplicated frontmatter field caused the dev server to crash Duplicated frontmatter field causes the dev server to crash Mar 17, 2023
@Princesseuh
Copy link
Member

It shouldn't crash for sure, but nonetheless duplicated keys are not supported in YAML, so they shouldn't work

@MoustaphaDev
Copy link
Member Author

Right, the crashing behavior is the actual issue

@matthewp matthewp self-assigned this Mar 23, 2023
@matthewp matthewp mentioned this issue Mar 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matthewp matthewp

Labels
- P3: minor bug An edge case that only affects very specific usage (priority)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Prevent frontmatter errors from crashing dev server withastro/astro
3 participants
@matthewp @Princesseuh @MoustaphaDev