{"push_id":1868334883,"size":215,"distinct_size":77,"ref":"refs/heads/master","head":"4c895ef9029182d0eb8adcdfc2706ecab9aaeba1","before":"f7f39453bdce36b3723ddb7012f2381c900e1761","commits":[{"sha":"06498563ded26506a042490e5a55f478b6c22eaf","author":{"name":"Dustin Kut Moy Cheung","email":"4097c986ac2f0513c6e9c0277cfce663e77c2bd5@redhat.com"},"message":"[NCL-2775] Weird date when closing a milestone\n\nThe issue was caused because `that.data.endDate` is null but\n`that.data.plannedEndDate` is generally not.\n\nThe fix is to check if `that.data.endDate` is null; if it is then use\nthe `that.data.plannedEndDate` as value, else use `that.data.endDate`.","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/06498563ded26506a042490e5a55f478b6c22eaf"},{"sha":"fb74bfaae5a769153393850a5784f7c886773939","author":{"name":"Dustin Kut Moy Cheung","email":"4097c986ac2f0513c6e9c0277cfce663e77c2bd5@redhat.com"},"message":"Remove secret mount from build agent pod def\n\nWe don't need that secret mount anymore. That mount contains TLS certs;\nhowever these certs are already mounted into the builder image pods, so\nit's not necessary to have the secret mounted.","distinct":true,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/fb74bfaae5a769153393850a5784f7c886773939"},{"sha":"2df3c420917db38991eb581e9fcdff250073c33f","author":{"name":"Dustin Kut Moy Cheung","email":"4097c986ac2f0513c6e9c0277cfce663e77c2bd5@redhat.com"},"message":"[NCL-2409][UI] Add ability to edit project desc","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/2df3c420917db38991eb581e9fcdff250073c33f"},{"sha":"993e6ec0b020296e50633b0f79e5ceffb8c3f47c","author":{"name":"Martin Kelnar","email":"500af55b4c9d506e99a04e31e0fad51adcff9358@redhat.com"},"message":"NCL-2716 Warning when default configuration is used","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/993e6ec0b020296e50633b0f79e5ceffb8c3f47c"},{"sha":"fae1cc31da10d1f77db8374ffacd1795689d9dee","author":{"name":"Jakub Bartecek","email":"1b5946260f8e265350bcf5a4112f8188218bc8bb@redhat.com"},"message":"[NCL-2898]: Fix NPE in /build-records/###/built-artifacts","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/fae1cc31da10d1f77db8374ffacd1795689d9dee"},{"sha":"f1abac6d6d29bb8c131c61f5fbcfe06ddb073286","author":{"name":"Matej Lazar","email":"00c5f180426a21872e400080298e6e6eaa36dcf8@gmail.com"},"message":"NCL-2590: Fix SystemErrorTest.","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/f1abac6d6d29bb8c131c61f5fbcfe06ddb073286"},{"sha":"503bfcf694e4dfe0fa5c664384655f5e1a04ce67","author":{"name":"Matej Lazar","email":"00c5f180426a21872e400080298e6e6eaa36dcf8@gmail.com"},"message":"Merge pull request #1459 from matejonnet/master\n\nNCL-2590: Fix SystemErrorTest.","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/503bfcf694e4dfe0fa5c664384655f5e1a04ce67"},{"sha":"879e59615f2fede3a811a6a10a6afcc61b5ba105","author":{"name":"Matej Lazar","email":"00c5f180426a21872e400080298e6e6eaa36dcf8@gmail.com"},"message":"NCL-2938 - Add Missing buildEndTime property to final event notifications.","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/879e59615f2fede3a811a6a10a6afcc61b5ba105"},{"sha":"7bfd6b48efc1238ac4cbd011edb3ffc88ca5fd0a","author":{"name":"Matej Lazar","email":"00c5f180426a21872e400080298e6e6eaa36dcf8@gmail.com"},"message":"Merge pull request #1460 from matejonnet/master\n\nNCL-2938 - Add Missing buildEndTime property to final event notificat…","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/7bfd6b48efc1238ac4cbd011edb3ffc88ca5fd0a"},{"sha":"078914d84bafe1ce12c84e57a090388357551480","author":{"name":"Jakub Bartecek","email":"1b5946260f8e265350bcf5a4112f8188218bc8bb@redhat.com"},"message":"Merge pull request #1458 from jbartece/artifactsNpe\n\n[NCL-2898]: Fix NPE in /build-records/###/built-artifacts","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/078914d84bafe1ce12c84e57a090388357551480"},{"sha":"141c556fece955ac06dc577aba01a5255c107431","author":{"name":"Martin Kelnar","email":"500af55b4c9d506e99a04e31e0fad51adcff9358@redhat.com"},"message":"NCL-2908 New build statuses added","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/141c556fece955ac06dc577aba01a5255c107431"},{"sha":"36e6e77c550c81077eac606ffc51290aff50113f","author":{"name":"Jakub Bartecek","email":"1b5946260f8e265350bcf5a4112f8188218bc8bb@redhat.com"},"message":"[NCL-1197]: Fix build of empty BuildConfigSet never ends","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/36e6e77c550c81077eac606ffc51290aff50113f"},{"sha":"0793f5ac7e1917ce73ba98467577dc0b110ab752","author":{"name":"Matej Lazar","email":"00c5f180426a21872e400080298e6e6eaa36dcf8@gmail.com"},"message":"Update REST error responses format.","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/0793f5ac7e1917ce73ba98467577dc0b110ab752"},{"sha":"3543562f5139480ad170b647c0acc26d3b376c10","author":{"name":"Matej Lazar","email":"00c5f180426a21872e400080298e6e6eaa36dcf8@gmail.com"},"message":"Report REST exceptions in JSON format.","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/3543562f5139480ad170b647c0acc26d3b376c10"},{"sha":"25f36a3bbd1154d9cfa4e9f6d70c68c5987ce92d","author":{"name":"Matej Lazar","email":"00c5f180426a21872e400080298e6e6eaa36dcf8@gmail.com"},"message":"Merge pull request #1463 from matejonnet/master\n\nNCL-2893 - Most of error responses from backend are not in expected format","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/25f36a3bbd1154d9cfa4e9f6d70c68c5987ce92d"},{"sha":"851237a1881ecc0d362447701efcca8a6c700e68","author":{"name":"Jakub Bartecek","email":"1b5946260f8e265350bcf5a4112f8188218bc8bb@redhat.com"},"message":"Merge pull request #1462 from jbartece/emptySet\n\n[NCL-1197]: Fix build of empty BuildConfigSet never ends","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/851237a1881ecc0d362447701efcca8a6c700e68"},{"sha":"56f900b78409b5a545a33ea730533f8accb40b3d","author":{"name":"Martin","email":"7a081349f21e2f66b1b363a1917c5ba02e2ca6e7@users.noreply.github.com"},"message":"Merge pull request #1461 from matedo1/ncl2908-waitingBuilds\n\nNCL-2908 New build statuses added","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/56f900b78409b5a545a33ea730533f8accb40b3d"},{"sha":"fe0fc1d61ec3ea2c005886e2e42afa7688c3ceb9","author":{"name":"Dustin Kut Moy Cheung","email":"4097c986ac2f0513c6e9c0277cfce663e77c2bd5@redhat.com"},"message":"[NCL-2954] Add opt to only allow some http methods\n\nAdd option to only allow some HTTP methods for a url. You can also\nspecify the port of the url.\n\nThe option is specified in pnc-config.json as such:\n\n```\n...\n\"firewallAllowedDestinations\": \"...\",\n\"allowedHttpOutgoingDestinations\": [\n    {\"url\": \"<url>\", allowedMethods: \"PUT,POST\"},\n    ...\n]\n```\n\nIf the same URL is specified in both `firewallAllowedDestinations` and\n`allowedHttpOutgoingDestinations`, the one in the latter has higher\npriority than the former. This is enforced in the startup script running\nin the builder pods.","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/fe0fc1d61ec3ea2c005886e2e42afa7688c3ceb9"},{"sha":"66193fea6eea0a8573857694a66ad724c215cf8b","author":{"name":"Dustin Kut","email":"b6bf58ae8ed5ca223e44533bb5a674c0c72f10da@users.noreply.github.com"},"message":"Merge pull request #1464 from thescouser89/ncl-2954\n\n[NCL-2954] Add opt to only allow some http methods","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/66193fea6eea0a8573857694a66ad724c215cf8b"},{"sha":"35e4af29ca920aee7b4c3258a324ab2ecc2ed54c","author":{"name":"Dustin Kut Moy Cheung","email":"4097c986ac2f0513c6e9c0277cfce663e77c2bd5@redhat.com"},"message":"Fix nested JSON statement for allowedHttpOutgoingDestinations\n\nSo we were putting a JSON string inside another JSON structure. However,\nwe forgot to properly escape the JSON string.\n\nThe situation we were getting into is:\n\n```\n{\n    \"name\": \"allowedHttpOutgoingDestinations\",\n    \"value\": \"[{\"url\":\"http://haha.com\",\"allowedMethods\":\"PUT\"}]\"\n},\n```\n\nThis is obviously wrong JSON. The value of the key 'value' should have\nits string escaped to produce valid JSON.\n\n```\n{\n    \"name\": \"allowedHttpOutgoingDestinations\",\n    \"value\": \"[{\\\"url\\\":\\\"http://haha.com\\\",\\\"allowedMethods\\\":\\\"PUT\\\"}]\"\n},\n\n```\n\nThis commit fixes that bug.","distinct":false,"url":"https://api.github.com/repos/alexcreasy/pnc/commits/35e4af29ca920aee7b4c3258a324ab2ecc2ed54c"}]}
{"actor":{"display_login":"alexcreasy"}}