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

[ReleasePrep][2018.11.05]RI Dev into master #6625

Merged
merged 22 commits into from
Nov 6, 2018
Merged

[ReleasePrep][2018.11.05]RI Dev into master #6625

merged 22 commits into from
Nov 6, 2018

Conversation

ryuyu
Copy link
Contributor

@ryuyu ryuyu commented Nov 5, 2018

No description provided.

stevenaw and others added 20 commits October 17, 2018 13:56
 Add GetEtagAsync.
* Set enableVersionHeader to 'false' everywhere

* Disable ASP.NET debug mode by default
* Package validation returns error and warning messages as IValidationMessage.
PacakgesController converts those to JsonValidationMessage for Json responses and passes through IValidationMessage to Views for rendering. All PackageController Json responses use JsonValidationMessage in case of failure for consistency.
ApiController always uses plain text version of the IValidationMessage.
Updated tests to accomodate for the changes.

* Initial view change

* Safety fallbacks for error communication to the upload page.

* Organized usings.

* Making sure `PackageShouldNotBeSignedUserFixableValidationMessage` does not emit raw HTML.

* Made type checks consistent

* may it

* PR feedback.
Hotfixes (do not CC sender for contact owners) to dev
* Typosquatting optimization: add caching

* update

* Move locker to cache class

* Refactor, Decouple and Update a lot of things

* Update

* Fix unit test

* Update again

* Set default cache expire time as 24 hours

* Some changes

* Use TimeSpan for expire time

* Add some tests for cache
@zhhyu zhhyu self-requested a review November 6, 2018 22:11
@ryuyu ryuyu merged commit bf7e98f into master Nov 6, 2018
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.

10 participants