{"push_id":2632216028,"size":73,"distinct_size":73,"ref":"refs/heads/4.1","head":"1f71fb2c58da6c187815648ccbefbc403505b0cd","before":"e5b6a1a401aad59d5de7108cb56f1a8c3607315b","commits":[{"sha":"48fe14402a98ed600fa14348984e688af725561c","author":{"name":"Trustin Lee","email":"8efd86fb78a56a5145ed7739dcb00c78581c5375@motd.kr"},"message":"Update os-maven-plugin to 1.6.0 (#7884)\n\nMotivation:\r\n\r\nA new version of os-maven-plugin has been released.\r\n\r\nModifications:\r\n\r\n- Update os-maven-plugin from 1.5.0 to 1.6.0\r\n\r\nResult:\r\n\r\n- No visible changes\r\n- Feels good","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/48fe14402a98ed600fa14348984e688af725561c"},{"sha":"1b1f7677acec6a5ae06a552adc4c046b2ce2490d","author":{"name":"Rikki Gibson","email":"1eae18db07d3092e87097fadc13b4f232593c91c@gmail.com"},"message":"Implement isWritable and ensureWritable on ReadOnlyByteBufferBuf (#7883)\n\nMotivation:\r\n\r\nIt should be possible to write a ReadOnlyByteBufferBuf to a channel without errors. However, ReadOnlyByteBufferBuf does not override isWritable and ensureWritable, which can cause some handlers to mistakenly assume they can write to the ReadOnlyByteBufferBuf, resulting in ReadOnlyBufferException.\r\n\r\nModification:\r\n\r\nAdded isWritable and ensureWritable method overrides on ReadOnlyByteBufferBuf to indicate that it is never writable. Added tests for these methods.\r\n\r\nResult:\r\n\r\nCan successfully write ReadOnlyByteBufferBuf to a channel with an SslHandler (or any other handler which may attempt to write to the ByteBuf it receives).","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/1b1f7677acec6a5ae06a552adc4c046b2ce2490d"},{"sha":"010dbe3c734aff7b400c7be8666e5f0c055fca62","author":{"name":"Robin Wang","email":"cbcad22bb9e0ad2bf870d852d6dee07fc795a3a8@gmail.com"},"message":"Optimize space usage of FastThreadLocal (#7861)\n\nMotivation:\r\nA FastThreadLocal instance currently occupies 2 slots of InternalThreadLocalMap of every thread. Actually for a FastThreadLocalThread, it does not need to store cleaner flags of FastThreadLocal instances. Besides, using BitSet to store cleaner flags is also better for space usage.\r\n\r\nModification:\r\nUse BitSet to optimize space usage of FastThreadLocal.\r\n\r\nResult:\r\nAvoid unnecessary slot occupancy. Cleaner flags are now stored into a BitSet.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/010dbe3c734aff7b400c7be8666e5f0c055fca62"},{"sha":"b47fb817991b42ec8808c7d26538f3f2464e1fa6","author":{"name":"Norman Maurer","email":"475492d9e56d4e9f737fa652c1d3f22ec1130ed7@googlemail.com"},"message":"EventLoop.schedule with big delay fails (#7402)\n\nMotivation:\r\n\r\nUsing a very huge delay when calling schedule(...) may cause an Selector error when calling select(...) later on. We should gaurd against such a big value.\r\n\r\nModifications:\r\n\r\n- Add guard against a very huge value.\r\n- Added tests.\r\n\r\nResult:\r\n\r\nFixes [#7365]","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/b47fb817991b42ec8808c7d26538f3f2464e1fa6"},{"sha":"eaf177133638b528a8d644b7d0bf3665cd90a18c","author":{"name":"Norman Maurer","email":"475492d9e56d4e9f737fa652c1d3f22ec1130ed7@googlemail.com"},"message":"Don't use VM.maxDirectMemory() on z/OS to retrieve direct memory limit. (#7886)\n\nMotivation:\r\n\r\nOn z/OS netty initializes this value with 64M, even the direct accessible memory is actually unbounded.\r\n\r\nModifications:\r\n\r\nSkip usage of VM.maxDirectMemory() on z/OS.\r\n\r\nResult:\r\n\r\nMore correct direct memory limit detection. Fixes https://github.com/netty/netty/issues/7654.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/eaf177133638b528a8d644b7d0bf3665cd90a18c"},{"sha":"f4d7e8de140048047299808bb7e707c15342b826","author":{"name":"Norman Maurer","email":"475492d9e56d4e9f737fa652c1d3f22ec1130ed7@googlemail.com"},"message":"Respect MAX_MESSAGES_PER_READ in LocalChannel / LocalServerChannel. (#7885)\n\nMotivation:\r\n\r\nLocalChannel / LocalServerChannel did not respect read limits and just always read all of the messages.\r\n\r\nModifications:\r\n\r\n- Correct respect MAX_MESSAGES_PER_READ settings\r\n- Add unit tests\r\n\r\nResult:\r\n\r\nFixes https://github.com/netty/netty/issues/7880.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/f4d7e8de140048047299808bb7e707c15342b826"},{"sha":"b818852cdb0ee56ce3cf939e2faa538d519baabd","author":{"name":"Devrim Şahin","email":"eadd3b64a329e2b17eea5803930ca22efc10d949@gmail.com"},"message":"Update DatagramPacket.recipient() to return the actual destination IP (#7879)\n\nMotivation:\r\n\r\nDatagramPacket.recipient() doesn't return the actual destination IP, but the IP the app is bound to.\r\n\r\nModification:\r\n\r\n- IP_RECVORIGDSTADDR option is enabled for UDP sockets, which allows retrieval of ancillary information containing the original recipient.\r\n- _recvFrom(...) function from transport-native-unix-common/src/main/c/netty_unix_socket.c is modified such that if IP_RECVORIGDSTADDR is set, recvmsg is used instead of recvfrom; enabling the retrieval of the original recipient.\r\n- DatagramSocketAddress also contains a 'local' address, representing the recipient.\r\n- EpollDatagramChannel is updated to return the retrieved recipient address instead of the address the channel is bound to.\r\n\r\nResult:\r\n\r\nFixes #4950.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/b818852cdb0ee56ce3cf939e2faa538d519baabd"},{"sha":"c0ae5e697c3c4fdf81f0cfc199d18aacd3b8ecba","author":{"name":"Alexey Kachayev","email":"6f01c95906f90f1a93fd08bb66da46cad029cf95@gmail.com"},"message":"DnsAddressResolverGroup to use pluggable DnsNameResolverBuilder (#7793)\n\nMotivation:\r\n\r\nRight now to customize DNS name resolver when using DnsAddressResolverGroup\r\none should subclass implementation and override newNameResolver method when\r\nin fact it's possible to collect all settings in a DnsNameResolverBuilder\r\ninstance. Described in #7749.\r\n\r\nModifications:\r\n\r\n- Added new constructor for DnsNameResolverBuilder in order to delay\r\n  EventLoop specification\r\n\r\n- Added copy() method to DnsNameResolverBuilder to provide an immutable\r\n  copy of the builder\r\n\r\n- Added new single-argument constructor for DnsAddressResolverGroup and\r\n  RoundRobinDnsAddressResolverGroup accepting DnsNameResolverBuilder\r\n  instance\r\n\r\n- DnsAddressResolverGroup to build a new resolver using DnsNameResolverBuilder\r\n  given instead of creating a new one\r\n\r\n- Test cases to check that changing channelFactory after the builder was passed\r\n  to create a DnsNameResolverGroup would not propagate to the name resolver\r\n\r\nResult:\r\n\r\nMuch easier to customize DNS settings w/o subclassing DnsAddressResolverGroup","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/c0ae5e697c3c4fdf81f0cfc199d18aacd3b8ecba"},{"sha":"dfeb4b15b587bf5298555e969a01378853efab8f","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Fix IllegalReferenceCountException when using Http2MultiplexCodec and a DefaultHttp2GoAwayFrame with a non empty ByteBuffer is received. (#7894)\n\nMotivation:\r\n\r\nWe incorrectly called frame.release() in onHttp2GoAwayFrame which could lead to IllegalReferenceCountExceptions.  The call of release() is inappropriate because the fireChannelRead() in onHttp2Frame() will handle it.\r\n\r\nModifications:\r\n\r\n- Not call frame.release()\r\n- Add a unit test\r\n\r\nResult:\r\n\r\nFxies https://github.com/netty/netty/issues/7892.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/dfeb4b15b587bf5298555e969a01378853efab8f"},{"sha":"b5fde3e0cdddfa96d65e85c3699e7ac8f0fd3ebc","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Add #ifdef statements so the compilation of the native code not fails on MacOS (#7895)\n\nMotivation:\r\n\r\nb818852cdb0ee56ce3cf939e2faa538d519baabd added support for IP_RECVORIGDSTADDR but did not include any #ifdef statements to ensure its usable at all (which is not the case on MacOS).\r\n\r\nModifications:\r\n\r\nAdd #ifdef statements to check for IP_RECVORIGDSTADDR.\r\n\r\nResult:\r\n\r\nCompilation works again on MacOS.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/b5fde3e0cdddfa96d65e85c3699e7ac8f0fd3ebc"},{"sha":"0261e0066216358b8acd390da96c85aaa8713b91","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Allow to call AbstractDiskHttpData.setContent(Unpooled.EMPTY_BUFFER) multiple times. (#7890)\n\nMotivation:\r\n\r\nIt should be possible to call setContent(Unpooled.EMPTY_BUFFER) multiple times just like its possible to do the same with a non empty buffer.\r\n\r\nModifications:\r\n\r\n- Correctly reset underlying storage if called multiple times.\r\n- Add tests\r\n\r\nResult:\r\n\r\nFixes https://github.com/netty/netty/issues/6418","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/0261e0066216358b8acd390da96c85aaa8713b91"},{"sha":"bd800fa7e7065dd74125f34298b69bb279f4be8e","author":{"name":"Daniel Schobel","email":"afc5e40f7e0ce50d0caf1701580b337509dfc04f@gmail.com"},"message":"Add null-check to Htt2FrameCodec#consumeBytes. (#7899)\n\nMotivation:\r\n\r\nStreams can be deregistered so we can't assume their existence in the stream map.\r\n\r\nModifications:\r\n\r\nAdd a null-check in case a stream has been deregistered.\r\n\r\nResult:\r\n\r\nFixes #7898.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/bd800fa7e7065dd74125f34298b69bb279f4be8e"},{"sha":"c60263e8a3336aae182fdde2dd0202a667cba880","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Correctly handle responses with status 205 and payload. (#7891)\n\nMotivation:\r\n\r\nHTTP responses with status of 205 should not contain a payload. We should enforce this.\r\n\r\nModifications:\r\n\r\nCorrectly handle responses with status 205 and payload by set Content-Length: 0 header and stripping out the content.\r\n\r\nResult:\r\n\r\nFixes https://github.com/netty/netty/issues/7888","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/c60263e8a3336aae182fdde2dd0202a667cba880"},{"sha":"095dadcbec7cd056ea1c0947d57d9084fae6d897","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Fix race in ChannelReadHandler used during LocalChannel testing. (#7904)\n\nMotivation:\r\n\r\nChannelReadHandler is used in tests added via f4d7e8de140048047299808bb7e707c15342b826. In the handler we verify the number of messages we receive per read() call but missed to sometimes reset the counter which resulted in exceptions.\r\n\r\nModifications:\r\n\r\nCorrectly reset read counter in all cases.\r\n\r\nResult:\r\n\r\nNo more unexpected exceptions when running LocalChannel tests.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/095dadcbec7cd056ea1c0947d57d9084fae6d897"},{"sha":"a598c3b69b55f930b91a8265f42d1a3cceb75ddd","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Enforce sane upper limit for TTL in DefaultDnsCache. (#7907)\n\nMotivation:\r\n\r\nIn b47fb817991b42ec8808c7d26538f3f2464e1fa6 we limited the max supported delay to match what our internal implementat can support. Because of this it was possible that DefaultDnsCache produced an IllegalArgumentException when it tried to schedule a expiration > 3 years.\r\n\r\nModifications:\r\n\r\nLimit the max supported TTL to 2 years which is safe for all our EventLoop implementations.\r\n\r\nResult:\r\n\r\nNo more exceptions when adding records to the cache with a huge TTL.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/a598c3b69b55f930b91a8265f42d1a3cceb75ddd"},{"sha":"b83328606fdb7782eae7f26425a189f1a99bfbac","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"DefaultDnsCache should store more then one Entry per hostname. (#7906)\n\nMotivation:\r\n\r\nDue a bug we did never store more then one address per hostname in DefaultDnsCache.\r\n\r\nModifications:\r\n\r\n- Correctly store multiple entries per hostname\r\n- Add tests\r\n\r\nResult:\r\n\r\nDefaultDnsCache correctly stores more then one entry. Also fixes https://github.com/netty/netty/issues/7882 .","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/b83328606fdb7782eae7f26425a189f1a99bfbac"},{"sha":"358249e5c9b3f832ae88b3ae9cf9f692106d4cc6","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Allow to disable native transport and native ssl support via system property. (#7903)\n\nMotivation:\r\n\r\nSometimes it's useful to disable native transports / native ssl to debug a problem. We should allow to do so with a system property so people not need to adjust code for this.\r\n\r\nModifications:\r\n\r\nAdd system properties which allow to disable native transport and native ssl.\r\n\r\nResult:\r\n\r\nEasier to disable native code usage without code changes.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/358249e5c9b3f832ae88b3ae9cf9f692106d4cc6"},{"sha":"d03bd44e9d79e8b73eea137c2e0373ed0a06f31e","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Add test for caching failed queries in DefaultDnsCache. (#7909)\n\nMotivation:\r\n\r\nWe had no test that validated the handling of caching failures for DefaultDnsCache.\r\n\r\nModifications:\r\n\r\nAdd testcase.\r\n\r\nResult:\r\n\r\nMore tests FTW.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/d03bd44e9d79e8b73eea137c2e0373ed0a06f31e"},{"sha":"54c3de0f8aa9e4eb2c95a368dcaa0b8cbb48403b","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Add test to validate SSLEngine does not zero out src buffer on wrap. (#7914)\n\nMotivation:\r\n\r\nWe had a bug-report that claimed the src buffer used by OpenSslEngine will be zero out.\r\n\r\nModifications:\r\n\r\nAdd testcase to ensure correct behaviour\r\n\r\nResult:\r\n\r\nTestcase for https://github.com/netty/netty/issues/7753","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/54c3de0f8aa9e4eb2c95a368dcaa0b8cbb48403b"},{"sha":"18b170bd7ad91153692af6830b7b1afb13835638","author":{"name":"Norman Maurer","email":"330e4303411e1fbd4ff825c4416851d277ea7a5f@apple.com"},"message":"Cleanup resolver-dns test code. (#7916)\n\nMotivation:\r\n\r\nWe had some stuff that could be cleaned up in the resolver-dns test classes.\r\n\r\nModifications:\r\n\r\nCleanup code.\r\n\r\nResult:\r\n\r\nCleaner code.","distinct":true,"url":"https://api.github.com/repos/OverNull/netty/commits/18b170bd7ad91153692af6830b7b1afb13835638"}]}
{"actor":{"display_login":"OverNull"}}