{"push_id":1239769982,"size":42,"distinct_size":2,"ref":"refs/heads/maint","head":"13e24c1a21010b9660e60f2e1725863e78d807b4","before":"75d6a1e18827b1c0251acc0e879ae37b47b413d5","commits":[{"sha":"fc62a9f51e34aaf7a2b1c6c8b3895d1612eb742a","author":{"name":"mruzicka","email":"e0a498bcf6aaadd0f51732f79cb5e12689330378@gmail.com"},"message":"Merge pull request #130 from parisiale/maint\n\n(maint) Increase TTL of integration test msgs to 5 s","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/fc62a9f51e34aaf7a2b1c6c8b3895d1612eb742a"},{"sha":"c09aa18be9a9cceaf44c6ef7fa9d3f7e8ca99ad9","author":{"name":"Michael Smith","email":"cb62b41feb3fb2c295d70e250fca81bcfe6bd095@puppet.com"},"message":"(packaging) Prepare for version 0.7.2","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/c09aa18be9a9cceaf44c6ef7fa9d3f7e8ca99ad9"},{"sha":"e9ff5cfc8df526428da2292b2c7ea2bbb5056623","author":{"name":"Alessandro Parisi","email":"1e47da2b769e75491a098bc6c3b20fcdb575918c@users.noreply.github.com"},"message":"Merge pull request #131 from MikaelSmith/release-0.7.2\n\n(packaging) Prepare for version 0.7.2","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/e9ff5cfc8df526428da2292b2c7ea2bbb5056623"},{"sha":"2db73b94d24173f72ecbdc20367cd7fc42f21ec6","author":{"name":"Jenkins CI","email":"ea125c00642277991785d4f46d514c725076a465@puppetlabs.com"},"message":"Version 0.7.2","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/2db73b94d24173f72ecbdc20367cd7fc42f21ec6"},{"sha":"bfa3f4277f194dcb8b7b1422c87bb67997d2c4a9","author":{"name":"Jenkins CI","email":"ea125c00642277991785d4f46d514c725076a465@puppetlabs.com"},"message":"Version 0.7.3-SNAPSHOT","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/bfa3f4277f194dcb8b7b1422c87bb67997d2c4a9"},{"sha":"3ca577db77252553ed1a886576166e511c9e19ef","author":{"name":"parisiale","email":"005be46eca2fe6c0c380646fcfef904f8c8472bd@puppetlabs.com"},"message":"(PCP-516) CapsuleLog's sender may not be a PCP URI\n\nAs the Message schema does not guarantee that :sender is a PCP URI, we\ncannot assume that in capsule/summarize, so we define CapsuleLog shcema\nwith :sender being a string.\n\nAdding a few unit tests for the capsule namespace.","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/3ca577db77252553ed1a886576166e511c9e19ef"},{"sha":"3e5835a0a3d4a5387c25e7227ecad89058e1dd33","author":{"name":"Michael Smith","email":"cb62b41feb3fb2c295d70e250fca81bcfe6bd095@puppet.com"},"message":"Merge pull request #132 from parisiale/PCP-516\n\n(PCP-516) CapsuleLog's sender may not be a PCP URI","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/3e5835a0a3d4a5387c25e7227ecad89058e1dd33"},{"sha":"35fd9502929f28073884aa56848186030cf9dfcd","author":{"name":"mruzicka","email":"e0a498bcf6aaadd0f51732f79cb5e12689330378@gmail.com"},"message":"(PCP-487) Increase timeouts in tests seriously (#133)\n\nThis commit increases expiry timeouts on PCP messages exchanged during\r\ntests by a factor of 5. This is to avoid occasional failures in CI\r\nwe keep seeing.\r\nIn contrast to previous attempts to do the same this commit doesn't\r\nchange the actual timeouts but rather redefines the function which\r\napplies the timeout values to the PCP messages:\r\n`puppetlabs.pcp.message/set-expiry`.\r\nThis has the desired effect of raising the timeouts for both the test\r\ncode and the tested code at the same time (previous attempts were only\r\nraising the timeouts in the test code).\r\nAdditionally, timeout is also raised in the\r\n `puppetlabs.pcp.testutils.client/recv!`\r\nfunction which, too, was causing test failures by not waiting long\r\nenough for messages to be received.\r\nFinally the `it-closes-connections-when-not-running-test` test is\r\nredesigned such that it avoids another instance of not waiting long\r\nenough for the broker to respond.","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/35fd9502929f28073884aa56848186030cf9dfcd"},{"sha":"2f4ed66826984424029df9697a73bf76a040bcba","author":{"name":"parisiale","email":"005be46eca2fe6c0c380646fcfef904f8c8472bd@puppetlabs.com"},"message":"(PCP-524) Add error_message data's id when available\n\nWith this commit the broker adds the id of the message that triggered\nand error when replying with an error_message.","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/2f4ed66826984424029df9697a73bf76a040bcba"},{"sha":"ce00cf79e41c2270820c0e6224a39c626037422e","author":{"name":"Michael Smith","email":"cb62b41feb3fb2c295d70e250fca81bcfe6bd095@puppet.com"},"message":"Merge pull request #135 from parisiale/PCP-524\n\n(PCP-524) Add error_message data's id when available","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/ce00cf79e41c2270820c0e6224a39c626037422e"},{"sha":"032310570a63e1af181aa1d5307a4549d081e962","author":{"name":"parisiale","email":"005be46eca2fe6c0c380646fcfef904f8c8472bd@puppetlabs.com"},"message":"(PCP-528) Fix unit and integration lein profiles\n\n[skip ci]","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/032310570a63e1af181aa1d5307a4549d081e962"},{"sha":"77c52d58ef3e11668e8dbfd2f678dd6a418a861c","author":{"name":"Michael Smith","email":"cb62b41feb3fb2c295d70e250fca81bcfe6bd095@puppet.com"},"message":"Merge pull request #136 from parisiale/PCP-528\n\n(PCP-528) Fix unit and integration lein profiles","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/77c52d58ef3e11668e8dbfd2f678dd6a418a861c"},{"sha":"609c1437a8af59d72b517c60bb7bf85266988774","author":{"name":"parisiale","email":"005be46eca2fe6c0c380646fcfef904f8c8472bd@puppetlabs.com"},"message":"(PCP-526) Add (dev, test)-schema-validation profiles\n\nAdding new profiles that extend, respectively, the dev and test\nprofiles by using an :injection entry to enable schema validation\nvia (schema.core/set-fn-validation! true).\n\nRefactoring dependencies of test profiles into new `test-base` profile.\n\nAdding new `test-all` task that executes all tests with and without\nschema verification.","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/609c1437a8af59d72b517c60bb7bf85266988774"},{"sha":"c7ef261aaa10bb2fe7601f167ea9290428f4ea29","author":{"name":"parisiale","email":"005be46eca2fe6c0c380646fcfef904f8c8472bd@puppetlabs.com"},"message":"(PCP-526) Don't turn on schema validation in test sources","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/c7ef261aaa10bb2fe7601f167ea9290428f4ea29"},{"sha":"290ae0407af5eeb24d083ff3336825d3727cacac","author":{"name":"parisiale","email":"005be46eca2fe6c0c380646fcfef904f8c8472bd@puppetlabs.com"},"message":"(PCP-526) Don't test error when wrapping a bad Message\n\nAfter disabling schema validation, capsule/wrap does not enforce that\nthe Message argument is a valid Message, so we can't test that anymore.\n\nNote that with PCP-525 we will ensure that only valid PCP messages are\naccepted by pcp-broker; we can safely drop the Message validation when\ncreating a Capsule.","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/290ae0407af5eeb24d083ff3336825d3727cacac"},{"sha":"2bf937f0d56ac46f271d51dcc5ff63af4dba30ba","author":{"name":"parisiale","email":"005be46eca2fe6c0c380646fcfef904f8c8472bd@puppetlabs.com"},"message":"(PCP-526) Don't test schema validation in pcp-broker transition\n\nSchema validation has been disabled by default; don't rely on it when\ntesting core/determine-next-state behaviour.","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/2bf937f0d56ac46f271d51dcc5ff63af4dba30ba"},{"sha":"8af9f9b07237786f2c4db9626d68228dd3bb8cf1","author":{"name":"Michael Smith","email":"cb62b41feb3fb2c295d70e250fca81bcfe6bd095@puppet.com"},"message":"Merge pull request #138 from parisiale/PCP-526_lein\n\n(PCP-526) Don't do schema validation in tests by default","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/8af9f9b07237786f2c4db9626d68228dd3bb8cf1"},{"sha":"c70502c35d8c7bed2ccf1037652047b6dc65f7b0","author":{"name":"parisiale","email":"005be46eca2fe6c0c380646fcfef904f8c8472bd@puppetlabs.com"},"message":"(maint) Style changes","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/c70502c35d8c7bed2ccf1037652047b6dc65f7b0"},{"sha":"f67acacc3fd34b01d99231f37f3726f2d3c34479","author":{"name":"parisiale","email":"005be46eca2fe6c0c380646fcfef904f8c8472bd@puppetlabs.com"},"message":"(PCP-489) Improve onMessage handling\n\nChecking the validity of incoming messages only once, meaning:\n - ignore messages other than associate_request before Association is\n   complete;\n - authentication;\n - authorization;\n - TTL check.\n\nRemoving the transitions map that keept track of the state of the\nassociation during the message double dispatch mechanism in favour of\nmaking explicit the processing workflow.\n\nAvoiding to update the 'connections' map at each processed message. In\npractice, the onMessage handler will now update the map only if the\nConnection instance being processed is actually updated; that happens\nonly in case of a successful associate_request.\n\nEnsure incoming messages are valid Message instances by explicitly\nvalidating against the Message schema; this is necessary as we don't\nhave schema validation enabled on function calls.\n\nRemoving schema validation calls for data content created by the broker.\nFactoring out functions that create data content objects in order to\nenable schema validation only when globally set (with appropriate lein\nprofile).\n\nIn case of invalid (as above) associate_request, an unsuccessful\nassociate_response is sent back to the requester and the WebSocket\nconnection is closed. That's a behaviour change, as:\n - in case of authentication failure, the broker used to send a PCP\n   error and close the WebSocket;\n - in case of authorization failure, the broker used to only close the\n   WebSocket.\n\nImproving documentation.\n\nUpdating tests and creating new ones.\n\nUsing the async 'put!' call in testutils/client.clj as the previous\n'>!!' blocking function was creating a deadlock in those cases where the\ntest client was expecting both PCP and WebSocket responses from the\nbroker under test.","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/f67acacc3fd34b01d99231f37f3726f2d3c34479"},{"sha":"08c582ce5a7d89c8287eb1b6f8e4eb5d93987935","author":{"name":"Michael Smith","email":"cb62b41feb3fb2c295d70e250fca81bcfe6bd095@puppet.com"},"message":"Merge pull request #139 from parisiale/PCP-489_improvements\n\n(PCP-489) Improve onMessage handling","distinct":false,"url":"https://api.github.com/repos/parisiale/pcp-broker/commits/08c582ce5a7d89c8287eb1b6f8e4eb5d93987935"}]}
{"actor":{"display_login":"parisiale"}}