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.

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

tag-check.js has race condition with cuStaleness.js #216

Closed
roysmith opened this issue Apr 13, 2022 · 0 comments
Closed

tag-check.js has race condition with cuStaleness.js #216

roysmith opened this issue Apr 13, 2022 · 0 comments
Labels
bug Something isn't working tag-check Related to tag-check.js

Comments

@roysmith
Copy link
Owner

If tag-check runs before cuStaleness, the M or P tag gets parsed as part of the username, causing cuStaleness to make API calls like this:

action: query
format: json
list: users|logevents
usprop: registration
ususers: M_Chutrandi
letype: newusers
letitle: User:M_Chutrandi
@roysmith roysmith added bug Something isn't working tag-check Related to tag-check.js labels Apr 13, 2022
roysmith added a commit that referenced this issue Apr 13, 2022
@roysmith
Move sock tag outside of span.cuEntry node.
49fe2ea 
Fixes #216.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working tag-check Related to tag-check.js
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@roysmith