rails.git - Mirror of official rails repo with custom fixes.

	Commit message (Collapse)	Author	Age	Files	Lines
...
\| * \| \| \| \| \| \| \| \| \| \|	Path parameters should default to UTF8	eileencodes	2017-08-01	5	-14/+44
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This commit changes the behavior such the path_params now default to UTF8 just like regular parameters. This also changes the behavior such that if a path parameter contains invalid UTF8 it returns a 400 bad request. Previously the behavior was to encode the path params as binary but that's not the same as query params. So this commit makes path params behave the same as query params. It's important to test with a path that's encoded as binary because that's how paths are encoded from the socket. The test that was altered was changed to make the behavior for bad encoding the same as query params. We want to treat path params the same as query params. The params in the test are invalid UTF8 so they should return a bad request. Fixes #29669 Eileen M. Uchitelle, Aaron Patterson, & Tsukuru Tanimichi
\| * \| \| \| \| \| \| \| \| \| \|	Merge pull request #30029 from yhirano55/fix_rails_command_comments	Sean Griffin	2017-08-01	1	-3/+3
\| \|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Fix rails_command comments
\| \| * \| \| \| \| \| \| \| \| \| \|	[ci skip] Fix rails_command comments	Yoshiyuki Hirano	2017-08-01	1	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30021 from roberts1000/fix-rubocop-issues	Rafael França	2017-08-01	2	-3/+3
\| \|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix rubocop issues
\| \| * \| \| \| \| \| \| \| \| \| \| \|	Fix rubocop style issues in yarn and spring.rb templates"	roberts1000	2017-07-31	2	-3/+3
\| \| \| \|/ / / / / / / / / / \| \| \|/\| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30024 from kamipo/remove_redundant_unless_branch	Sean Griffin	2017-07-31	1	-2/+0
\| \|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove redundant `unless current_adapter?(:OracleAdapter)`
\| \| * \| \| \| \| \| \| \| \| \| \| \|	Remove redundant `unless current_adapter?(:OracleAdapter)`	Ryuta Kamizono	2017-07-28	1	-2/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Talk about bytes not characters	Rafael Mendonça França	2017-07-31	1	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Closes #30012
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Add backticks around method [ci skip]	Ryuta Kamizono	2017-08-01	1	-1/+1
\| \| \|/ / / / / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	And make reference to `Relation`.
\| * \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30017 from bslobodin/fix-typo-in-assert-changes	Rafael França	2017-07-31	1	-1/+1
\| \|\ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix typo in assert_changes error message
\| \| * \| \| \| \| \| \| \| \| \| \| \|	fix typo in assert_changes error message	Boris Slobodin	2017-07-31	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Prevent interpolation of ERB elements	David Heinemeier Hansson	2017-08-01	1	-6/+6
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Add the storage.yml file by default	David Heinemeier Hansson	2017-08-01	1	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Follow same format as action_cable	David Heinemeier Hansson	2017-08-01	1	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Cut down on installation advice	David Heinemeier Hansson	2017-08-01	1	-8/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Drop README terms no longer relevant	David Heinemeier Hansson	2017-08-01	1	-12/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Add require	David Heinemeier Hansson	2017-08-01	1	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	We moved to storage.yml	David Heinemeier Hansson	2017-08-01	2	-35/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Rely on top-level files	David Heinemeier Hansson	2017-08-01	2	-150/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30034 from ffmike/patch-1	Guillermo Iguaran	2017-08-01	1	-0/+8
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add Azure configuration to Active Storage template
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Add Azure configuration to Active Storage template	Mike Gunderloy	2017-08-01	1	-0/+8
\|/ / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	With Active Storage supporting Azure, we should show people how to set it up. Incorporates the minor changes originally in https://github.com/rails/activestorage/pull/88 as well
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30030 from claudiob/only-disk	Guillermo Iguaran	2017-08-01	1	-1/+3
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Let ActiveStorage tests pass only for Disk
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Let ActiveStorage tests pass only for Disk	claudiob	2017-08-01	1	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	If you have a "service/configurations.yml" file, but every single line is commented out, then an error occurs when running tests: ``` git:active-storage-import~/code/rails/activestorage$ rake ~/code/rails/activestorage/test/test_helper.rb:17:in `<top (required)>': undefined method `deep_symbolize_keys' for false:FalseClass (NoMethodError) from ~/code/rails/activestorage/test/controllers/direct_uploads_controller_test.rb:1:in `require' ``` The reason is that `YAML.load(..an empty file content..)` simply returns `false`, and not `{}`. This PR fixes this behavior so tests can also run when no remote service is available.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30032 from claudiob/remove-httparty	Guillermo Iguaran	2017-08-01	5	-21/+18
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove HTTParty dependency
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Don't depend on HTTParty	claudiob	2017-08-01	5	-21/+18
\|/ / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	"httparty" is only added in #30020 to write two tests to make PUT requests against S3 and GCS. The same requests can be made with net/http, removing a dependency from the Gemfile.
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Setup travis to be able to run CI tests against S3	David Heinemeier Hansson	2017-07-31	5	-33/+33
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Move the direct_upload: true convenience option from the activestorage ↵	David Heinemeier Hansson	2017-07-31	3	-20/+10
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	helper into actionview
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Move to new storage.yml location	David Heinemeier Hansson	2017-07-31	1	-0/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Directory and config installation now done by app_generator	David Heinemeier Hansson	2017-07-31	1	-7/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Create the skeleton directories needed and ignore accordingly	David Heinemeier Hansson	2017-07-31	2	-0/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix gitignore to be relative	David Heinemeier Hansson	2017-07-31	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Follow form from actioncable	David Heinemeier Hansson	2017-07-31	1	-1/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Load activestorage by default	David Heinemeier Hansson	2017-07-31	2	-0/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Use config/storage.yml as part of the skeleton and loading	David Heinemeier Hansson	2017-07-31	9	-43/+48
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Resolve gemspec typo	David Heinemeier Hansson	2017-07-31	2	-1/+86
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	No longer used	David Heinemeier Hansson	2017-07-31	1	-23/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Require activestorage per default with asset pipeline	David Heinemeier Hansson	2017-07-31	1	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Bring activestorage dependencies into the general Gemfile	David Heinemeier Hansson	2017-07-31	3	-233/+14
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Initialize changelog	David Heinemeier Hansson	2017-07-31	1	-0/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Use standard Rails layout for gemspec	David Heinemeier Hansson	2017-07-31	1	-12/+24
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Include default storage configuration	David Heinemeier Hansson	2017-07-31	5	-0/+36
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Add 'activestorage/' from commit '3f4a7218a4a4923a0e7ce1b2eb0d2888ce30da58'	Rafael Mendonça França	2017-07-31	121	-0/+7370
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	git-subtree-dir: activestorage git-subtree-mainline: 0d58e7e478e79c2d6b2a39a4444d2a17a903b2a6 git-subtree-split: 3f4a7218a4a4923a0e7ce1b2eb0d2888ce30da58
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Azure Storage support (#36)	Dino Maric	2017-07-31	7	-0/+198
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Microsoft Azure storage support * Add support for Microsoft Azure Storage * Comply with the new headers implementation
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Depend on Ruby >=2.2.2 (#85)	Dino Maric	2017-07-31	2	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	activestorage.js 0.1.1	Javan Makhmali	2017-07-30	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Configure per-service request headers for direct uploads (#83)	Javan Makhmali	2017-07-30	11	-20/+81
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Configure per-service request headers for direct uploads * Fix header hashes
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Convert type inquiry into root predicates for base types alone instead	David Heinemeier Hansson	2017-07-28	2	-16/+14
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Must always return the options even if they werent converted	David Heinemeier Hansson	2017-07-28	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Add Blob#type as a StringInquirer	David Heinemeier Hansson	2017-07-28	2	-0/+20
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Add a direct_upload: true option to file_field_tag and Form#file_field	David Heinemeier Hansson	2017-07-28	2	-1/+19
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Then users don't have to bother with manually referring to internal routes.