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

Check if edit relates to existing endpoint #114

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

Conversation

agrrh
Copy link

@agrrh agrrh commented Apr 18, 2020

Hey there!

There is note in sixnines.rb file about possibility to create new endpoint without payment:
https://github.com/yegor256/sixnines/blob/8bdbd70/sixnines.rb#L346-L353

I was able to reproduce this by using following request:

http --form POST https://www.sixnines.io/a/edit \
  'Cookie:glogin=3YBx....jYvJ' \
  old='https://example.org' \
  new='https://agrrh.com'

Result is new endpoint was created:
Screenshot from 2020-04-18 22-24-07

BTW, I was trying to remove endpoint afterwards, but it stays there w/o any error messages. Feel free to wipe my account.

@0crat
Copy link
Collaborator

0crat commented Apr 18, 2020

@agrrh/z this pull request is too small, just 8 lines changed (less than 10), there will be no formal code review, see §53 and §28; in the future, try to make sure your pull requests are not too small; @null/z please review this and merge or reject

@agrrh
Copy link
Author

agrrh commented Jun 15, 2020

@yegor256 May I ask you to take a look at this PR?

@agrrh
Copy link
Author

agrrh commented Mar 31, 2021

@yegor256
Sorry for mentioning you directly, but I'm afraid my website is still 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.

2 participants