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

Fix things #17

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

Fix things #17

wants to merge 46 commits into from

Conversation

subliun
Copy link
Owner

@subliun subliun commented Apr 27, 2016

No description provided.

@SkyzohKey
Copy link

Wow I known ToxMe code was shit but not at this point. Could have been using templates, at least. e.e

... to prevent nonexistant records from showing up and blocking addition of a user.
- Prevent already deleted items from showing up in results (fixes inability to delete and re-add the same name without the need to restart toxme)
Only allow root to be crawled.
This is mostly for people who have some kind of request related to having their old account removed after they forgot the password.
This is for people who want to change their account but don't remember their password.

Note that you will always need to be able to prove the account you want to modify is yours. In case of losing the toxme.io password, this means you must be able to log into this account and add me from it.
Add a more apparent way of contacting me
@@ -0,0 +1,3 @@
User-agent: *
Allow: /$

Choose a reason for hiding this comment

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

I'm pretty sure the $ mark is only useful in the router regex, marking the end of the line.

bennabiy and others added 6 commits December 27, 2016 12:34
pip3 update in multiple locations
libsodium is no longer available for direct download so we compile it manually.
Update README.md to reflect missing dependency
Make it more apparent because people were apparently confused by it.
@toxme.io is not an e-mail domain
TheDcoder and others added 15 commits April 1, 2017 10:17
Previously, full tox id shows only when user had bio. Makes no sense to do it this way.
Show full tox ID even when user has no bio
Sick and tired of people adding me to "test toxme". I'm here for technical issues.
Add echobot as a test contact instead of myself.
No more blank favicon on the pain page of toxme.
looking glass+tox padlock. I think it fits.
Add favicon to the template
squashed changes to remove reference to bios from public view

* Omit Bios from public view

Hello darkness my ol' friend.

* more fixes

@LittleVulpix, squash this so I don't have to git clone
* add_ui year bump

* addkeyweb_success year bump

* api_error_pretty year bump

* edit_ui year bump

* fourohfour year bump

* lookup_home year bump

* onemomentplease year bump

* public_userlist year bump
People constantly add me "no I just want to test toxme". The link to echobot for "TESTING TOXME" is literally right above mine! So there, no direct contact easily visibly on the site now. Enjoy your echobot.
@SkyzohKey
Copy link

About the latest change, maybe make a [email protected] or [email protected] address

* Remove "not in use" line.

* Update logic to disable only friend listing

When cfg value is set to 0, it now still allows adding/editing but no more public list of any user; you can still query normally however.
Could be done with the CFG but really, I don't wanna redirect to that page at all.
All the files updated with new copyright year.
https://blog.fefe.de/?ts=a3695410 and this is why we can't have nice things.
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.

6 participants