Skip to content
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

modify GALX because of google login page change. #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

monkeydone
Copy link

No description provided.

@JamesBradbury
Copy link

I suggest that the following may be more robust to future changes:

galx = re.compile('<input name="GALX" [^>]+ value="(?P<galx>[a-zA-Z0-9_-]+)">')

@monkeydone
Copy link
Author

it’s good!but you give me the regular,i’m not test success! I use the follow!
galx = re.compile('.\s_name="GALX"\s.\s*value="(?P[a-zA-Z0-9-]+)">')

在 2014年4月9日,下午4:46,JamesBradbury [email protected] 写道:

I suggest that the following may be more robust to future changes:

galx = re.compile(']+ value="(?P[a-zA-Z0-9_-]+)">')


Reply to this email directly or view it on GitHub.

@monkeydone
Copy link
Author

your regular is very good!

@JamesBradbury
Copy link

Thanks, I think it may be been shown wrongly at first because I didn't understand how to post code into GitHub comments.

@monkeydone
Copy link
Author

o,your code is right! I see it at the github. And your regular is beautiful!


在 2014年4月9日,下午5:32,JamesBradbury [email protected] 写道:

Thanks, I think it may be been shown wrongly at first because I didn't understand how to post code into GitHub comments.


Reply to this email directly or view it on GitHub.

@mrkristofer
Copy link

I'm getting back "Exception("Could not find requested section")". Any suggestions or solutions out there?

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