Skip to content

Conversation

josevalim
Copy link
Contributor

Regexes cannot be part of module bodies in Erlang/OTP 28, so this PR adjusts the repository accordingly.

A new release with this fix would be appreciated, since Erlang/OTP 28 has just been released. :)

@solnic solnic self-requested a review May 21, 2025 11:13
Copy link
Collaborator

@whatyouhide whatyouhide left a comment

Choose a reason for hiding this comment

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

😭

@whatyouhide whatyouhide merged commit 97d62ed into getsentry:master May 21, 2025
4 checks passed
@whatyouhide whatyouhide mentioned this pull request May 22, 2025
solnic added a commit that referenced this pull request May 27, 2025
solnic added a commit that referenced this pull request May 27, 2025
solnic added a commit that referenced this pull request May 28, 2025
* Tweak credit card regex handling for OTP-28

Just like in #889

* Update CHANGELOG
@grzuy
Copy link

grzuy commented Jul 14, 2025

Thanks @josevalim for the fix!

It still not possible to set this option now on config/*.exs module bodies to a list of regex, right?

Docs still point user to set this to a list of Regex.t() in https://hexdocs.pm/sentry/Sentry.html#module-configuration

image

@josevalim
Copy link
Contributor Author

We are planning to fix it on Erlang/OTP 28.1.

firestack added a commit to mbta/prediction_analyzer that referenced this pull request Jul 24, 2025
this fails to compile on OTP 28 (getsentry/sentry-elixir#889)
But this also relies on Phoenix 1.6. so we need to update that first
firestack added a commit to mbta/prediction_analyzer that referenced this pull request Jul 25, 2025
this fails to compile on OTP 28 (getsentry/sentry-elixir#889)
But this also relies on Phoenix 1.6. so we need to update that first
firestack added a commit to mbta/prediction_analyzer that referenced this pull request Jul 25, 2025
this fails to compile on OTP 28 (getsentry/sentry-elixir#889)
But this also relies on Phoenix 1.6. so we need to update that first
firestack added a commit to mbta/prediction_analyzer that referenced this pull request Jul 25, 2025
this fails to compile on OTP 28 (getsentry/sentry-elixir#889)
But this also relies on Phoenix 1.6. so we need to update that first
firestack added a commit to mbta/prediction_analyzer that referenced this pull request Aug 13, 2025
this fails to compile on OTP 28 (getsentry/sentry-elixir#889)
But this also relies on Phoenix 1.6. so we need to update that first
firestack added a commit to mbta/prediction_analyzer that referenced this pull request Aug 29, 2025
this fails to compile on OTP 28 (getsentry/sentry-elixir#889)
But this also relies on Phoenix 1.6. so we need to update that first
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.

3 participants