Is there an existing issue for this? <li class="

DNS / BIND9 Zone files are wrongly highlighted (`err` class) about chroma HOT 2 CLOSED

tarneaux commented on June 9, 2024

DNS / BIND9 Zone files are wrongly highlighted (`err` class)

from chroma.

Comments (2)

alecthomas commented on June 9, 2024 1

FYI you can test without the Go toolchain by downloading one of the release binaries and running it with:

chroma -l ./dns.xml <input>

from chroma.

tarneaux commented on June 9, 2024

After a bit of trial and error, it seems like the following regex has just the right scope to detect only domain names:

([\w\d\.-]+\.[a-z-]{2,24}\.?|^(@|[\w\d-]+))

I have only tested it on a regex playground though, and this is why I am not submitting a PR yet (setting up the working environment is a bit of a hassle with go for me).

Git diff:

$ git diff origin/master
diff --git a/lexers/embedded/dns.xml b/lexers/embedded/dns.xml
index 5ac3c25..ef8f663 100644
--- a/lexers/embedded/dns.xml
+++ b/lexers/embedded/dns.xml
@@ -27,7 +27,7 @@
       <rule pattern="\b([\w\d.-])+\.($|\s)">
         <token type="NameProperty"/>
       </rule>
-      <rule pattern="^(@|[\w\d-]+)">
+      <rule pattern="([\w\d\.-]+\.[a-z-]{2,24}\.?|^(@|[\w\d-]+))">
         <token type="NameClass"/>
       </rule>
       <rule pattern="^\$(TTL|GENERATE|INCLUDE|ORIGIN)">

from chroma.

Recommend Projects