-
Notifications
You must be signed in to change notification settings - Fork 200
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
RsWithCookie must reject if invalid characters #46
Comments
@yegor256 got it, tag |
@yegor256 thanks for the report, I topped your acc for 15 mins, payment ID |
@popprem this task is yours, please help (see our key principles of work). If you have any technical questions, don't hesitate to ask right here The budget here is 30 mins, which is exactly how much time will be paid for, when the task is completed |
davvd / Yegor, Sorry for the delay. I'm in the process of shifting to a new house with family. I will start working on this project from Monday (30th of March 2015). Thanks. |
@popprem the ticket is with you for 15 days already. If it is not finished in the next 48 hours, it will be re-assigned to someone else, see No Obligations principle, this article should help if you're stuck. -30 added to your rating, current score is: -30 |
@davvd Sorry, got internet connection yesterday only. |
@popprem you should use forks and pull requests, see http://www.yegor256.com/2014/04/15/github-guidelines.html |
@yegor256 How do you execute local build? Earlier i did "mvn package", was successful... checked in the changes but Appveyor failed with checkstyle issues. Now i tried to execute build with com.qulice:qulice-maven-plugin:check goal, but getting loads of checkstyle issues in many classes. |
@popprem check README.md, there is an explanation |
@popprem you've been working with this task for too long. I'm sorry, but I'll assign someone else. Please stop working with it. See our no obligations principle... -60 added to your rating, at the moment it is: -90 |
@yegor256 Understood, thanks. |
@yegor256. @popprem you can not wait for me. You can do next: set global .gitattributes file(download it from my commits, "git config --global core.attributesfile file_path") -it will apply to all repositories, where .gitattributes file not exists. Next time when you clone (may for checkout too, I am not sure) git will apply correct end line char (\n) to repository, and files on them and gulice will be happy. |
@YamStranger Thanks, applied the same. |
@dmzaytsev please go ahead with this task, it's yours |
added unit tests
@dmzaytsev many thanks! |
@yegor256 since there is no milestone yet I set it to "1.0" |
@elenavolokhova please, let us know what do you think about this ticket, according to our QA rules |
According to our quality rules:
As far as you are the performer of this ticket you are responsible for its quality. Please make sure that everyone follows this rule in your tickets in the future. Ask them to fix issues before closing the ticket. Please confirm that everything is clear and you have no questions regarding this approach. |
@elenavolokhova everything is clear. I confirm |
@davvd Ticket passed QA review. Quality is acceptable. |
@elenavolokhova thanks for the review, we'll try better next time |
@dmzaytsev I added 10 mins to @elenavolokhova (for QA review) in transaction 55835838. Much appreciated! 1 hour was added to your account, payment ID |
RsWithCookie
must reject (throw runtime exception) when its ctor is called with name or value containing invalid (unprintable) characters. Let's create a test reproducing this situation and also let's document what RFC we're using to check for character validity.The text was updated successfully, but these errors were encountered: