search

tiredofit / docker-traefik-cloudflare-companion

Automatically Create CNAME records for containers served by Traefik
MIT License
384 stars 64 forks source link

Fixed invalid escape sequence errors #107

Open juchong opened 2 months ago

juchong commented 2 months ago

Two regex errors popped up in my logs recently. This pull request makes the rule a raw string so that the expressions are properly evaluated.

The error: cloudflare-companion | 2024-08-02.22:14:58 [STARTING] ** [traefik-cloudflare-companion] [1] Starting Traefik Cloudflare Companion cloudflare-companion | /usr/sbin/cloudflare-companion:289: SyntaxWarning: invalid escape sequence '\.' cloudflare-companion | if re.match('traefik.*?\.rule', prop): cloudflare-companion | /usr/sbin/cloudflare-companion:322: SyntaxWarning: invalid escape sequence '\.' cloudflare-companion | if re.match('traefik.*?\.rule', prop):

After the fix: cloudflare-companion | 2024-08-02.22:17:08 [STARTING] ** [traefik-cloudflare-companion] [1] Starting Traefik Cloudflare Companion cloudflared-tunnel | 2024-08-03T05:17:08Z INF Registered tunnel connection connIndex=2 connection=<ID> event=0 ip=<IP> location=<LOC>protocol=quic

johntdyer commented 1 day ago

this would be a nice merge :)