Log of the #fcrepo channel on chat.freenode.net

Using timezone: Eastern Standard Time
* awoods leaves00:30
<dbernstein>Please don’t forget to join us for tomorrow’s meeting: https://wiki.duraspace.org/display/FF/2018-02-15+-+Fedora+Tech+Meeting00:45
* dbernstein leaves00:59
* bryjbrown joins08:15
* bryjbrown leaves08:29
* dhlamb joins08:31
* kefo joins08:42
* dwilcox joins09:00
* bryjbrown joins09:17
* awoods joins09:35
* peichman joins10:19
* github-ff joins10:20
[fcrepo4] bseeger opened pull request #1283: Check RDF mimetype in ebucore:hasMimeType triple (master...check_rdf_mimetype) https://git.io/vAnGk
* github-ff leaves
* peichman leaves10:26
* peichman joins10:27
* peichman leaves10:28
* dbernstein joins10:48
* whikloj joins
* dbernstein leaves10:52
* dbernstein joins
<whikloj>https://wiki.duraspace.org/display/FF/2018-02-15+-+Fedora+Tech+Meeting10:59
* peichman joins11:05
<whikloj>dbernstein++ # good job on the release11:06
* kestlund joins11:22
* awead joins11:24
* awead leaves
* bseeger joins11:27
* dbernstein leaves11:39
* kestlund leaves11:45
* kestlund joins
* peichman leaves12:01
* kestlund leaves12:05
* kestlund joins12:06
* kestlund leaves12:16
* kestlund joins12:33
* kestlund leaves12:46
* github-ff joins12:56
[fcrepo4] dbernstein deleted 4.7.5-maintenance at 1c28c07: https://git.io/vAnrW
* github-ff leaves
* github-ff joins12:57
[fcrepo4] dbernstein deleted 4.7.4-RC at 433680a: https://git.io/vAnrR
* github-ff leaves
* github-ff joins
[fcrepo4] dbernstein deleted 4.5.2-RC at 64a5159: https://git.io/vAnrz
* github-ff leaves
* github-ff joins13:01
[fcrepo-module-auth-rbacl] dbernstein opened pull request #54: Finalize release by merging 4.7.5-maintenance into 4.7-maintenance (4.7-maintenance...4.7.5-maintenance) https://git.io/vAnok
* github-ff leaves
* github-ff joins13:03
[fcrepo-module-auth-webac] dbernstein opened pull request #81: Finalize 4.7.5 release by merging 4.7.5-maintenance into 4.7-maintenance (4.7-maintenance...4.7.5-maintenance) https://git.io/vAnor
* github-ff leaves
* github-ff joins13:04
[fcrepo-mint] dbernstein opened pull request #10: Finalize 4.7.5 release by merging 4.7.5-maintenance into 4.7-maintenance (4.7-maintenance...4.7.5-maintenance) https://git.io/vAnoS
* github-ff leaves
* github-ff joins13:06
[fcrepo-audit] dbernstein deleted 4.7.5-RC at dd7a9a5: https://git.io/vAnoh
* github-ff leaves
* github-ff joins13:07
[fcrepo-audit] dbernstein opened pull request #49: Finalize 4.7.5 release by merging 4.7.5-maintenance into 4.7-maintenance (4.7-maintenance...4.7.5-maintenance) https://git.io/vAnKU
* github-ff leaves
* github-ff joins13:10
[fcrepo-webapp-plus] dbernstein deleted 4.7.5-RC at ba89ff9: https://git.io/vNVvf
* github-ff leaves
* github-ff joins
[fcrepo-webapp-plus] dbernstein opened pull request #63: Finalize 4.7.5 release by merging 4.7.5-maintenance into 4.7-maintenance (4.7-maintenance...4.7.5-maintenance) https://git.io/vAnKi
* github-ff leaves
* github-ff joins13:12
[fcrepo-mint] dbernstein deleted 4.7.2-RC at 3b15889: https://git.io/vAnKN
* github-ff leaves
* github-ff joins13:13
[fcrepo-mint] dbernstein deleted 4.5.2-RC at d245c55: https://git.io/vAnKp
* github-ff leaves
* github-ff joins
[fcrepo-mint] dbernstein deleted rc-4.5.0 at 72b81e6: https://git.io/vAn6f
* github-ff leaves
* github-ff joins13:18
[fcrepo4-vagrant] dbernstein opened pull request #87: Finalize 4.7.5 release by merging into master (master...4.7.5-RC) https://git.io/vAn6D
* github-ff leaves
* kestlund joins14:06
* dwilcox leaves14:22
* kestlund leaves14:28
<whikloj>bseeger: ping15:01
<bseeger>whikloj: I have to run to a meeting - I'll be back around 4:30 eastern time15:02
<whikloj>ok15:03
* dwilcox joins15:05
* dwilcox leaves15:09
* github-ff joins15:31
[fcrepo4-vagrant] whikloj pushed 1 new commit to master: https://git.io/vAnNW
fcrepo4-vagrant/master c1a02be dbernstein: Finalize 4.7.5 release by merging into master (#87)...
* github-ff leaves
* dwilcox joins15:35
* github-ff joins16:05
[fcrepo4] dbernstein pushed 1 new commit to gh-pages: https://git.io/vAnhV
fcrepo4/gh-pages 8063635 Daniel Bernstein: Update index.html for 4.7.5 release
* github-ff leaves
* dwilcox leaves16:14
<bseeger>whikloj: ping — I'm back around16:25
<whikloj>bseeger: hey, it was just around the two exceptions in your PR. It seems fine, just wondering why we are testing for IllegalArgumentException. Seems like that would be caught and wrapped in a ConstraintViolationException16:27
https://github.com/fcrepo4/fcrepo4/pull/1283/files#r168593034
<bseeger>yeah, it is a little odd, but does end up being a 400 error at the clients side.16:28
<whikloj>bseeger: yeah, which is why I'm not super concerned16:29
bseeger: is that because both exceptions are caught by the same ExceptionMapper?16:30
<bseeger>whikloj - one sec, looking for a code path16:33
<whikloj>np
<bseeger>this is the sparql patch path: https://github.com/fcrepo4/fcrepo4/blob/master/fcrepo-http-api/src/main/java/org/fcrepo/http/api/FedoraLdp.java#L50416:36
that will throw the InvalidArgumentException - get caught by the BadRequest and shipped back.
I could have it throw a Constraint Violation Exception… I think.
but not sure if it's worth it
<whikloj>bseeger: nah its fine, in the end it is the same16:37
* github-ff joins16:38
[fcrepo4] whikloj pushed 1 new commit to master: https://git.io/vAcfi
fcrepo4/master 99ce788 B. Seeger: Check RDF mimetype in ebucore:hasMimeType triple (#1283)...
* github-ff leaves
<bseeger>whikloj: do you use islandora 7 where you work?16:44
<whikloj>bseeger: yep http://digitalcollections.lib.umanitoba.ca
<bseeger>whikloj: cool - nice site16:45
<whikloj>bseeger: thanks, just don't try it on a mobile device. We haven't gotten there yet. :)16:46
* travis-ci joins16:52
fcrepo4/fcrepo4#5306 (master - 99ce788 : B. Seeger): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/80ada4ae5380...99ce788e0332
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/342081230
* travis-ci leaves
* bseeger leaves17:07
* kefo leaves17:39
* bryjbrown leaves17:41
* whikloj leaves18:00
* dhlamb leaves18:27
* bryjbrown joins18:49
* bryjbrown leaves19:09
* bryjbrown joins20:38
* bryjbrown leaves21:43