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

Suspend automatic @icon check and resume acct removal in the interim #1324

Merged
merged 1 commit into from
Feb 9, 2018

Conversation

Martii
Copy link
Member

@Martii Martii commented Feb 9, 2018

  • Apparently there is a long standing issue with EPROTO and this. Will need to dive in way deep and wait until it is fixed.
  • Project error trapping yielded no results
  • Altered to use request and same issue as denoted on the issue
  • Prevents the server trip from node builtin and other mentioned package.

Applies to #1323 and post #1303

* Apparently there is a long standing issue with EPROTO and this. Will need to dive in way deep and wait until it is fixed.
* Project error trapping yielded no results
* Altered to use *request* and same issue as denoted on the issue
* Prevents the server trip from *node* builtin and other mentioned package.

Applies to OpenUserJS#1323 and post OpenUserJS#1303
@Martii Martii added tracking upstream Waiting, watching, wanting. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. needs mitigation Needs additional followup. labels Feb 9, 2018
@Martii Martii merged commit d15672c into OpenUserJS:master Feb 9, 2018
@Martii Martii deleted the Pull-1303postNap branch February 9, 2018 07:16
@Martii
Copy link
Member Author

Martii commented Feb 9, 2018

Please note this code is not dead just suspended... we can't have ppl intentionally tripping the server. Rejection is much nicer than account removal for being bad with huge @icons... so back to the old method until it can be retested.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. needs mitigation Needs additional followup. tracking upstream Waiting, watching, wanting.
Development

Successfully merging this pull request may close these issues.

1 participant