Skip to content

Makes the route resolver pattern compatible with MicroPython's re #67

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Makes the route resolver pattern compatible with MicroPython's re #67

wants to merge 1 commit into from

Conversation

@Gattag
Copy link

@Gattag Gattag commented Apr 10, 2021

MicroPython does not support named classes within [...] (i.e. no [\\w] or [\\s] )

Fixes #38
@2e0byo
Copy link

2e0byo commented Sep 12, 2021

Was just going to PR for the same problem...

However I don't understand [\w.]. Since . matches anything, what's is \w doing? Doubtless something as I've no great experience with re.
@Gattag
Copy link
Author

Gattag commented Sep 13, 2021 •
edited
Loading

However I don't understand [\w.]. Since . matches anything, what's is \w doing? Doubtless something as I've no great experience with re.

When . is inside a character class [...], it not longer matches anything and only matches the literal . character
@tf131
Copy link

tf131 commented Sep 22, 2021

works fine, thank you
@2e0byo
Copy link

2e0byo commented Sep 22, 2021

However I don't understand [\w.]. Since . matches anything, what's is \w doing? Doubtless something as I've no great experience with re.

When . is inside a character class [...], it not longer matches anything and only matches the literal . character

Duh! Thank you. I was being daft.
klitos-giannak
klitos-giannak reviewed Jan 8, 2022
View reviewed changes
MicroWebSrv2/webRoute.py
raise Exception
argNames.append(argName)
regex += '/([\\w.]*)'
regex += '/([A-Za-z0-9_.]*)'
Copy link

@klitos-giannak klitos-giannak Jan 8, 2022

Choose a reason for hiding this comment

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

A suggestion here. Can we add support for whitespace as well? In order to support links like
http://example.com/some page that would be translated from web browsers with a %20
I have tested in CPython to add \s and works but if micropython can't support \w I am not sure how to handle it
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants

@Gattag @2e0byo @tf131 @klitos-giannak