{"push_id":875726878,"size":24,"distinct_size":24,"ref":"refs/heads/master","head":"3ab14cb1ad71e1254b025bf4ae721c3fed52a16f","before":"ed750d299573b15e4bc3e906e6c8e417ce064ee6","commits":[{"sha":"cb4532282ce32793c0ae1daf483383a73fe4e889","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Bump the \"down\" dependency to \">= 1.0.3\"\n\nThe Down gem had two bugs prior to this version. First one was that it\nwasn't working with the pretty_location plugin. Second one was that it\nwasn't preserving the file extension. So we bump up the required version\nso that we ensure that people will use a good version.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/cb4532282ce32793c0ae1daf483383a73fe4e889"},{"sha":"e49660723cfc46e085be2a7d8874ca9beddb9b21","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Don't require :max_size option in direct_upload\n\nLimiting the maximum file size should ultimately be set on the frontend\nserver (nginx, apache). This functionality can by bypassed anyway by\nuploading the file directly in the form, so this option is only to\nfortify the defenses a bit. We will probably make a `constraints` plugin\nin the future to ensure no attacks get through.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/e49660723cfc46e085be2a7d8874ca9beddb9b21"},{"sha":"3456844828cbe21aa049cff3fb5f2e5ee478c79a","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Remove the delete_invalid plugin from website","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/3456844828cbe21aa049cff3fb5f2e5ee478c79a"},{"sha":"0656864575b0987a33faf9b993db5909c9151f21","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Add module_include plugin","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/0656864575b0987a33faf9b993db5909c9151f21"},{"sha":"8eac431a231edbd9ec9d422cf20e1f401c7cfc36","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Add default_url_options plugin","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/8eac431a231edbd9ec9d422cf20e1f401c7cfc36"},{"sha":"c91d5fbed9b5ea20f8a1b4378798e4a392fbe3d4","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Move supporting JSON columns into sequel plugin\n\nAlthough it doesn't increase the amount of lines for the base\nfunctionality, it does increase complexity of those lines. Since I can't\nthink of any other case why this would be useful, it's better to move it\ninto the sequel plugin.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/c91d5fbed9b5ea20f8a1b4378798e4a392fbe3d4"},{"sha":"2d3eef0d2d375697e68aca4fa828606b9ac92295","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Log number of input processed files, not output\n\nSince we're logging the word \"PROCESS\", it makes much more sense to say\nhow many files we are PROCESS-ing, instead of how many files were the\nresult of processing.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/2d3eef0d2d375697e68aca4fa828606b9ac92295"},{"sha":"340c64515ed2a5533684bff5c1127f00977a005f","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Remove Shrine.delete\n\nThis method is redundant, and requires more code in the base\nfunctionality as well as other plugins. You should always delete files\nwith a `Shrine` instance, it doesn't matter with which storage it is\ninitialized. This of breaks backwards compatibility, but hopefully not\nmany people were using it, and it's better to do it now when releasing\n1.0 than to have it exist through 1.x versions.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/340c64515ed2a5533684bff5c1127f00977a005f"},{"sha":"d3b216a9d8a9cfd85989a401b984c7ed404011a1","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Move module_include plugins to correct category","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/d3b216a9d8a9cfd85989a401b984c7ed404011a1"},{"sha":"874ce733106103f5b17fc23fdadbb4dc483f8851","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Remove Shrine.io!\n\nThat method was only actually added for internal use, and I cannot\nimagine it being useful for public interface. It's ok to remove it\nbecause it's highly unlikely anyone was using it, and because we're\nmaking a major release.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/874ce733106103f5b17fc23fdadbb4dc483f8851"},{"sha":"2a60064b9b61286e897267371917ed5a02d9b38a","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Add SimpleCov","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/2a60064b9b61286e897267371917ed5a02d9b38a"},{"sha":"e307b24effc3298a3c5ead0efbd96b5af8e044af","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Remove the `max_size` option from direct_upload","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/e307b24effc3298a3c5ead0efbd96b5af8e044af"},{"sha":"3b1af85a2cf972b594a7bcfbab8c982686b54293","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Enable passing presign options in direct_upload\n\nOften users will want to configure presign options, in the direct_upload\nendpoint as well. One natural way seemed to allow passing query\nparameters for wanted presign options. But that imposes a security risk\nof people being able to configure all presign options (even\npermissions), and also it would create a Symbol DoS vulnerability,\nbecause we would have to convert these query parameters to symbols, and\nwe have no easy way of knowing the list of valid presign options.\n\nSo we instead add the ability to configure these options when loading\nthe plugin. We also yield the request object so that it's possible to\ngenerate different options depending on the request path and parameters.\n\nThis introduces a small backwards incompatibility, because we remove\nautomatic `content_type` assignment via a query parameter, and instead\nexpect the user to implement that. But I think it really makes sense,\nand we're releasing a major version anyway.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/3b1af85a2cf972b594a7bcfbab8c982686b54293"},{"sha":"e455e41cec3a977da45e990fbc4f2cb88c9a6ee1","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Remove JavaScript example in direct_upload docs\n\nThis can be in a separate guide, but I don't know to be recommending any\nparticular JS implementation, that's up to the user.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/e455e41cec3a977da45e990fbc4f2cb88c9a6ee1"},{"sha":"bf75ba6f3836aca989ddc01039a6af3cdccced61","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Fix tests and make them work on all Ruby versions\n\n1. On MRI `exit` effectively stops the require and continues on, but on\n   JRuby and Rubinius it stops the process. So instead we make the\n   general test suite exclude s3_test.rb by default, and running\n   s3_test.rb will still work without setting the ENV variable.\n\n2. We can now add Rubinius because it had the same problem.\n\n3. Current version of Rubinius has an bug where calling \"super\" with\n   default arguments which contain keyword arguments throws an error, so\n   we mitigate that bug and in those cases pass argument manually.\n\n4. In FileSystem test we forgot to actually initialize the storage with\n   permissions.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/bf75ba6f3836aca989ddc01039a6af3cdccced61"},{"sha":"8c2ca9e1a356cbe2344dbf87941c2df9fc76b134","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"SecureRandom.hex already returns a string\n\nFor some reason I was getting an error about this before, but probably\nbecause I was using a SecureRandom method which returns an integer.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/8c2ca9e1a356cbe2344dbf87941c2df9fc76b134"},{"sha":"3870d91d8327c51b006c72c19223fbafba7e92e2","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Fix S3#multi_delete and #clear! not using prefix\n\nWe also make sure we use aws-sdk version 2.1, because 2.2 throws\ndeprecation warnings, which we will fix once 3.0 comes out, but until\nthen we don't want warnings in our tests.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/3870d91d8327c51b006c72c19223fbafba7e92e2"},{"sha":"47484f9d9c7833b0bed228f8c0d887b7d33757de","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Note in the readme that we're supporting Rubinius","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/47484f9d9c7833b0bed228f8c0d887b7d33757de"},{"sha":"918e01582224b3c42c800d71580ab918f2106fb6","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Get the test suite working on Travis CI\n\n1. We specify latest Ruby versions because for example for\n   \"jruby-9.0.0.0\" Travis chose jruby-9.0.0.0.pre1 instead of the stable\n   version.\n\n2. We remove the filemagic test because it isn't worth the extra\n   overhead of installing filemagic-dev on Travis.\n\n3. We have to set RACK_ENV to \"development\" because Travis is setting it\n   to \"test\", which our logging plugin detects and disables logging.\n\n5. We make the s3_test.rb use a random prefix, because on Travis tests\n   are executing in parallel, and we want each of them to be inside\n   their own prefix.","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/918e01582224b3c42c800d71580ab918f2106fb6"},{"sha":"c18956c921be8e8ee036e83a2b88dc8422bb6a5a","author":{"email":"1e70908baa314345bbf67ecc18bb99bf93d99783@gmail.com","name":"Janko Marohnić"},"message":"Merge pull request #11 from janko-m/travis\n\nGet test suite working on Travis CI","distinct":true,"url":"https://api.github.com/repos/hakunin/shrine/commits/c18956c921be8e8ee036e83a2b88dc8422bb6a5a"}]}