{"push_id":922220326,"size":1000,"distinct_size":3,"ref":"refs/heads/integration-request-encoding-helpers","head":"52bb2d36d3141dcd8217221065d9b5fa2b12deba","before":"749b74b2df8009f579401a0590566964f5ea3bc0","commits":[{"sha":"748b2f9cb1ff05a8aab068e2fa802e9f12163e30","author":{"email":"daab9349e7897953586c7278844811677e111b79@gmail.com","name":"Kasper Timm Hansen"},"message":"Add enabled flag to the public file server.\n\nAs discussed in https://github.com/rails/rails/pull/19135#issuecomment-153385986.\n\nReplaces `serve_static_files` to unify the static options under the `public_file_server` wing.\n\nDeprecates `serve_static_files` accessors, but make them use the newer config internally.","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/748b2f9cb1ff05a8aab068e2fa802e9f12163e30"},{"sha":"08689a2d1f0165f353f62ef4626e87fe841ec735","author":{"email":"be8758dc4a66004612e74b567a662d4f1fbf4c2e@gmail.com","name":"Tijmen Brommet"},"message":"Add text template for source code\n\nWhen a request is made with AJAX and an error occurs, Rails will render\na text-template for the exception instead of the HTML error page\n(#11960).\n\nThe `.text.erb` variant of the `_source` template is currently missing,\ncausing HTML to be rendered in the response. This commit adds the text\ntemplate.\n\nTo keep the page scannable we only only show the first three source\nextracts.\n\nRelated to #14745.\n\nBefore:\n\n```\n~/testing-exceptions  ᐅ curl 'http://localhost:3000/' -H\n'X-Requested-With: XMLHttpRequest'\nRuntimeError in PostsController#index\n\n    <div class=\"source \" id=\"frame-source-0\">\n      <div class=\"info\">\n        Extracted source (around line <strong>#3</strong>):\n      </div>\n      <div class=\"data\">\n        <table cellpadding=\"0\" cellspacing=\"0\" class=\"lines\">\n          <tr>\n```\n\nAfter:\n\n```\n~/testing-exceptions  ᐅ curl 'http://localhost:3000/' -H\n'X-Requested-With: XMLHttpRequest'\nRuntimeError in PostsController#index\n\nExtracted source (around line #3):\n\n*3     raise\n```","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/08689a2d1f0165f353f62ef4626e87fe841ec735"},{"sha":"c8b24cc2675aceb020a8b8c8ee46854ab30d0125","author":{"email":"470bc578162732ac7f9d387d34c4af4ca6e1b6f7@metaskills.net","name":"Ken Collins"},"message":"Update docs/comments to use setup/teardown blocks vs. methods.\n\nAs discussed in the minitest-spec-rails issue (http://git.io/vlHxx) Rails uses setup/teardown callbacks. Defining `setup` or `teardown` methods vs. blocks will yield inconsistent behavior in the callback chain.","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/c8b24cc2675aceb020a8b8c8ee46854ab30d0125"},{"sha":"491f488f5ac4e50abfbd1a434971e41e48aa2f15","author":{"email":"84c66fc058fe65d1bb08ca33b21bfe0fa5cb8f17@gmail.com","name":"Rafael Mendonça França"},"message":"Fix deprecation message when frame doesn't have absolute_path\n\nWhen a frame is an eval block without filename argument there is no\nabsolute_path so the previous implementation would fail because `nil`\ndoesn't responds to `start_with?`.","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/491f488f5ac4e50abfbd1a434971e41e48aa2f15"},{"sha":"ab5fb4f22430afa58c334f7e7e142660164490e5","author":{"email":"99e6b749acbfbe2a596df99e91d24d6e1fdbee00@damienburke.com","name":"Damien Burke"},"message":"Don’t allow arbitrary data in back urls\n\n`link_to :back` creates a link to whatever was\npassed in via the referer header. If an attacker\ncan alter the referer header, that would create\na cross-site scripting vulnerability on every\npage that uses `link_to :back`\n\nThis commit restricts the back URL to valid\nnon-javascript URLs.\n\nhttps://github.com/rails/rails/issues/14444","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/ab5fb4f22430afa58c334f7e7e142660164490e5"},{"sha":"5a0e0e72995472e315738dcea5b5a12d6e3d3489","author":{"email":"7bbcc45b0f8f88a262415b9271579a59b891be2e@gmail.com","name":"Aaron Patterson"},"message":"don't start a new process for every test file\n\nwe can just reuse the parent process.  We should figure out what files\nare commonly required among the test files and try to require them in\nthe parent so that the require time cost is amortized across the\nprocesses.","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/5a0e0e72995472e315738dcea5b5a12d6e3d3489"},{"sha":"2acf6ac13f8bc3eb56619725e6185b94365071f0","author":{"email":"67694853bf7ec18265cf9799549f3660b4e0121a@unboxedconsulting.com","name":"Andrew White"},"message":"Fix spelling error [ci skip]","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/2acf6ac13f8bc3eb56619725e6185b94365071f0"},{"sha":"e38aa9d7464c6cfd851dd4436b67a858e4e899f3","author":{"email":"67694853bf7ec18265cf9799549f3660b4e0121a@unboxedconsulting.com","name":"Andrew White"},"message":"Remove incorrect comments\n\nColumns are no longer stored in an attribute since b8a533d.\n\n[ci skip]","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/e38aa9d7464c6cfd851dd4436b67a858e4e899f3"},{"sha":"351a663e57f2f20b3cd43b9ed751430665463659","author":{"email":"dc76e9f0c0006e8f919e0c515c66dbba3982f785@dballona.com","name":"Diego Ballona"},"message":"Fixing subscription callbacks","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/351a663e57f2f20b3cd43b9ed751430665463659"},{"sha":"91f5adf03aaf5347c694d901a81c25ee40525497","author":{"email":"470bc578162732ac7f9d387d34c4af4ca6e1b6f7@metaskills.net","name":"Ken Collins"},"message":"Update docs/comments to use setup/teardown blocks vs. methods.\n\n* Remove super for setup block. Not needed.","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/91f5adf03aaf5347c694d901a81c25ee40525497"},{"sha":"3f53df74d716c9db6b94a0ead9b8d2b526b6d8ee","author":{"email":"63b51a7b3ff1fafca81cda6f7ce2446b69bc82b3@gmail.com","name":"Yves Senn"},"message":"Merge pull request #22174 from metaskills/defsetup\n\nUpdate docs/comments to use setup/teardown blocks vs. methods. [ci skip]","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/3f53df74d716c9db6b94a0ead9b8d2b526b6d8ee"},{"sha":"df20d87c7b50cb7b19cc176a8f7944ee9f258446","author":{"email":"890cc9deb380ef7616b48a6d38cce799956aa21f@gmail.com","name":"Jeremy Daer"},"message":"Merge pull request #113 from dballona/fix-subscription-callbacks\n\nFixing subscription callbacks","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/df20d87c7b50cb7b19cc176a8f7944ee9f258446"},{"sha":"1e2f6bccc8bd2d5c8b40115c46bb4846cd673ed7","author":{"email":"63b51a7b3ff1fafca81cda6f7ce2446b69bc82b3@gmail.com","name":"Yves Senn"},"message":"formatting pass over Active Record changelog. [ci skip]","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/1e2f6bccc8bd2d5c8b40115c46bb4846cd673ed7"},{"sha":"e8b2c0535c203119e6733a982dc22734f3da1219","author":{"email":"84c66fc058fe65d1bb08ca33b21bfe0fa5cb8f17@gmail.com","name":"Rafael França"},"message":"Merge pull request #22180 from usertesting/arbitrary-url-in-link-to-back\n\nDon’t allow arbitrary data in back urls","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/e8b2c0535c203119e6733a982dc22734f3da1219"},{"sha":"7c1631fa48b8862f37d1026b4f0cf1061dd6947a","author":{"email":"8b69cfb584888a693f743cf62b7bb8c1f9b2aba0@gmail.com","name":"Pratik Naik"},"message":"Make sure cable closes the connection if open when responding to an invalid request","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/7c1631fa48b8862f37d1026b4f0cf1061dd6947a"},{"sha":"04bf3a174ed6a02b803ec91a6bd2be8f44137fd2","author":{"email":"e2c36f5fe0482fad7e2fa25be67845e57e37bc08@gmail.com","name":"Sean Linsley"},"message":"document `try!` in ActiveSupport core ext guide\n\nhttps://twitter.com/avdi/status/660141673993777156","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/04bf3a174ed6a02b803ec91a6bd2be8f44137fd2"},{"sha":"110e49179c8ceb9d4abd6110feec1496a3eac1f0","author":{"email":"84c66fc058fe65d1bb08ca33b21bfe0fa5cb8f17@gmail.com","name":"Rafael França"},"message":"Merge pull request #22182 from seanlinsley/patch-1\n\ndocument `try!` in ActiveSupport core ext guide","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/110e49179c8ceb9d4abd6110feec1496a3eac1f0"},{"sha":"a3e05f7c819ae7a3c08240f64a76439412d144cf","author":{"email":"daab9349e7897953586c7278844811677e111b79@gmail.com","name":"Kasper Timm Hansen"},"message":"Move `static_cache_contorl` deprecation changelog entry to Railties.\n\nThe configuration for `config.static_cache_control`, and its replacement\n`config.public_file_server.headers` are implemented in Railties.\n\nPeople would configure this in environment files, which is Railties domain too.","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/a3e05f7c819ae7a3c08240f64a76439412d144cf"},{"sha":"7b471dbf5bdab9579962e71bcf2e24e01bcc99ef","author":{"email":"daab9349e7897953586c7278844811677e111b79@gmail.com","name":"Kasper Timm Hansen"},"message":"Merge pull request #22173 from kaspth/enable-public-file-server\n\nAdd enabled flag to the public file server settings.","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/7b471dbf5bdab9579962e71bcf2e24e01bcc99ef"},{"sha":"a80fb6f9b3b553b59a486d7aa4673cc47397fab5","author":{"email":"63b51a7b3ff1fafca81cda6f7ce2446b69bc82b3@gmail.com","name":"Yves Senn"},"message":"tests, define `Rails.root` before loading Action Mailer.\n\nSince `bin/test` would define `Rails.root` before loading AM but `bundle\nexec rake` would define it after loading AM, this lead to the following\ntest failure when using `bin/test`:\n\n```\n--- expected\n+++ actual\n@@ -1 +1 @@\n-{:location=>\"/var/folders/_r/9kh50y4j0vn7zgg2p65z1lcm0000gn/T/mails\"}\n+{:location=>\"/Users/senny/Projects/rails/actionmailer/tmp/mails\"}\n```\n\nThis patch ensures that `Rails.root` is defined when loading AM to\nachieve consistent test results.","distinct":false,"url":"https://api.github.com/repos/kaspth/rails/commits/a80fb6f9b3b553b59a486d7aa4673cc47397fab5"}]}