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
email accepts ' which is not allowed #5473
Comments
The text you are pointing to literally says the opposite:
|
you're quoting the local, not domain part @dockimbel |
Certainly a Red Specs gray area. Today the lexer defines the rules, so some of those special chars don't work. e.g. The confusing part in this case is the tick mark normally used by lit-word!. As soon as you include The question, then, is should email allow ticks? If so, should it also allow them at the head? The only reason to do so is to support the full RFC spec, which we're not going to do. Too many conflicts with other types in Red. A tick at the head of an email looks confusing to me, reading like a literal word value. However, nobody is going to make you do that. It's OK as it is, IMO, but if we make ticks illegal, at least in the head position, I'm not opposed to that either. |
The provided code example has a single quote in the local part and not in the domain part (back-quote there). The ticket description is written in such a way that it confuses the reader about the reported issue... |
Describe the bug
Red accepts
'
in email which is not allowed as here (Thanks to @hiiamboris for the reference)(I don't know if it should be allowed to have illegal character to let Red "user" code to parse them or not)
To reproduce
Platform version
-----------RED & PLATFORM VERSION-----------
RED: [ branch: "master" tag: #v0.6.4 ahead: 5088 date: 16-Jan-2024/7:06:36 commit: #49a98d164a11da9e809a5aca45f462e662e5ca6d ]
PLATFORM: [ name: "Windows 11" OS: 'Windows arch: 'x86-64 version: 10.0.0 build: 22631 ]
The text was updated successfully, but these errors were encountered: