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

Using timezone: Eastern Standard Time
<dbernstein>Would anyone be willing to review this PR? https://github.com/fcrepo4/fcrepo4/pull/115601:19
* dbernstein leaves02:56
* coblej joins07:57
* kestlund joins08:18
* awoods joins08:23
* kestlund leaves08:31
* kestlund joins08:33
* dwilcox joins09:09
* jjtuttle leaves09:11
* jjtuttle joins09:12
* kestlund leaves09:27
* peichman joins09:28
* kestlund joins09:34
* kestlund1 joins09:37
* kestlund leaves09:40
* ajs6f joins09:49
* apb18 joins09:56
* th5 joins09:57
* coblej leaves10:03
* bseeger joins10:16
* coblej joins10:17
* awoods leaves10:19
* f4jenkins leaves
* anarchivist leaves
* cbeer leaves
* cbeer joins
* awoods joins10:22
* f4jenkins joins
* anarchivist joins10:31
* github-ff joins
[fcrepo4] awoods opened pull request #1157: Reduce logging level of WebApplicationException (master...fcrepo-2385) https://git.io/vMXFJ
* github-ff leaves
<barmintor>ajs6f: can I just agree vociferously that after about 2 dozen comments the new github thing was impenetrable and I just responded to the emails10:39
<ajs6f>barmintor: grumble grumble yeah exactly.
I feel like an old fart but I really do think it's an un-improvement
Also I actually am an old fart, so that's okay10:40
<barmintor>apb18 ajs6f I think y'all have a god summary there of the 3 big chnges, I'll add a commit before I head back to Queens today
<ajs6f>barmintor++++ #thanks for sticking with this and making it through10:41
<barmintor>ajs6f: did you also want me to be clear about the no param form being a delegation to the server?10:42
<apb18>barmintor: Fantastic!
<ajs6f>barmintor: I think it would be nice to say that. I think that the way you designed it, it's very much a feature, not just a chance amboguity10:43
it speaks to a desire we've heard from users
<barmintor>ajs6f: ok. the other point you suggested clarification would be good besides those 3 changes was...
* barmintor reading the comments threads
there was one more place you asked for things to be explicit10:44
<ajs6f>I think maybe the idea of servers SHOULD give informative bodies for all repsonses?
Was that it?
reading....
<awoods>nice work barmintor/ajs6f
<barmintor>Was it maybe a thing stating that Expect: 202-digest and Accept on the same request is outside the remit of the spec?10:45
* kestlund1 leaves
<ajs6f>barmintor: meaning what to return if either both fail or both succeed?10:46
barmintor: Or generally, when they interact ambiguously?
<barmintor>ajs6f: yeah, like "this request might not even make sense so that's brtween you and the impl"
* bseeger leaves
<barmintor>ajs6f: Rob Sanderson pushed back pretty strongly on permitting conneg on this interaction10:47
ajs6f: so maybe explicitly punting
<ajs6f>barmintor: Yeah, I think that's cool to say explcitly, "we can only help you so much. If you insist on jabbering unintelligibly on a street corner, we will just put some coins in your hand and walk sadly on."
<barmintor>right but in RFC language10:48
<ajs6f>eh, if that's what you like
<barmintor>we can have a parallel poetic spec
<ajs6f>The boardroom version and the street version.
There is one thing I don't like about disallowing conneg entirely-- there could be several helpful ways to give information about a failed request.10:49
Esepcially when we think about the complex forms of fixity for the link style.10:50
<barmintor>eek
<ajs6f>So I'd like to say, "The spec doesn't tell you how to deal with this" but not "the spec tells you not to do this".
<barmintor>the Accept applied to the entity on 4xx response? eek
yes, absolutely
<ajs6f>Yeah, that's all aI mean.
<barmintor>I am +1 that
<ajs6f>People who are smarter than us may figure out good way sto deal with this.
let's let them.
<barmintor>that what API v1.1 is for
<ajs6f>I thought everything had to be version 4 for all time?10:51
4 is the magic number.
<barmintor>that's fedora, the api is independently versioned :prayer emoji:
<ajs6f>I like to imagine Fedo Raadmin in a superhero costume with 4 on the chest, then he rips off the 4 and it's a 1 underneath.10:52
<shocked gasps>
* whikloj joins
<barmintor>yeah, the nice thing about not using 406 here is that we can explicitly say: This spec doesn't take a stance on what it means to combine conneg/Accept, and thus avoids use of 40610:53
y'all are wise
that is good10:54
* barmintor goes to a meeting
* github-ff joins10:55
[fcrepo4] escowles closed pull request #1157: Reduce logging level of WebApplicationException (master...fcrepo-2385) https://git.io/vMXFJ
* github-ff leaves
<ajs6f>barmintor++ # there goes a heck of a gunslinger
* travis-ci joins11:10
fcrepo4/fcrepo4#4896 (master - 3a5fbbb : Esmé Cowles): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/e41ac7ec68e8...3a5fbbb2f33e
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/193088372
* travis-ci leaves
* dhlamb_ joins11:22
* coblej leaves11:29
* apb18 leaves11:45
* coblej joins12:52
* peichman leaves12:55
* peichman joins12:57
* dbernstein joins12:58
* apb18 joins13:04
* peichman leaves13:31
* peichman joins13:34
* ajs6f leaves13:37
* ajs6f joins13:50
* ajs6f leaves13:55
* ajs6f joins14:11
* github-ff joins14:15
[fcrepo-import-export] escowles pushed 1 new commit to external-content: https://git.io/vM1re
fcrepo-import-export/external-content 5aefb5b Esmé Cowles: Code review
* github-ff leaves
* ajs6f leaves14:16
* ajs6f joins14:32
* ajs6f leaves14:37
* ajs6f joins14:43
* github-ff joins14:44
[fcrepo-import-export] awoods closed pull request #74: Adding option to retrieve and export external content (master...external-content) https://git.io/vMPmY
* github-ff leaves
* github-ff joins14:45
[fcrepo-import-export] awoods pushed 1 new commit to master: https://github.com/fcrepo4-labs/fcrepo-import-export/commit/9473d9d7f95eb6f3a6e551937e476bec5f125bff
fcrepo-import-export/master 9473d9d Esmé Cowles: Adding option to retrieve and export external content (#74)...
* github-ff leaves
<ajs6f>awoods: Small annoyance-- I am now getting emails of comments on Duraspace Jira tickets hours and hours after they are left.14:47
I wouldn't complain except it seems to be a new phenom. Any chance of a change that can be checked?
<awoods>ajs6f: is the issue one of getting the comments or that that are late?14:48
ajs6f: is the issue one of getting the comments or that they are late?
<ajs6f>I think that they are late.
awoods: Of course, it's hard to know if I'm not getting them :(
<awoods>ajs6f: based on a request from apb18 and ruebot last week, I changed the settings so that fcrepo-committers get notification for all Fedora JIRA updates.14:49
ajs6f: That is the only change I am aware of.14:50
<ajs6f>awoods: I wouldn't think that would have this effect...14:52
Well, leave it be and I will complain if it really becomes a prob.
<awoods>ajs6f: nor I
ajs6f: keep me posted. Also, it would be interesting to know if anyone else is seeing odd notification behavior.14:53
<barmintor>ajs6f apb18 added a commit that replaces 406 with 412, replaces 202-digest-link with a digest-link param on 202-digest, explicates default param digest selecton deferral, and explicates non-position on Accept/406 combined with Expect: 202-digest14:59
<ajs6f>barmintor: I'm on it. It sounds like we can merge that puppy.
<barmintor>ajs6f++15:00
<apb18>barmintor: Wonderful! At a quick glance, it looks great15:04
<ajs6f>Yep. I'm seeing a couple of places where I might want to tweak wording, but that is definitely a post-merge thing15:10
There is a point to be noted. As far as I can tell, this PR makes binary fixity fundamentally an optional service.
That may require some communication
* coblej leaves15:13
* coblej joins15:35
<ajs6f>barmintor: merging. I'll have a follow-on PR with some wordshopping15:38
acoburn: thanks for that issue close, forgot to do that15:44
<barmintor>ajs6f apb18 thanks for the careful read and contributions15:54
<ajs6f>barmintor: Always happy to fling my opinions at you like so many rotten fish.15:55
<barmintor>I feel like I could sleep for 3 days15:56
<apb18>barmintor: I do not deserve thanks, but you deserve 3 days of sleep!15:57
<ajs6f>barmintor: I thought they call it the city that never sleeps?15:58
<barmintor>ajs6f: on the other hand, it's also the city where you can't get anywhere by 9am, go figure
<ajs6f>badummp-ching!
gots to run16:00
* ajs6f leaves
* kestlund joins16:19
* apb18 leaves16:31
* kestlund leaves16:36
* dhlamb_ leaves16:45
* dhlamb_ joins16:57
* dhlamb_ leaves17:06
* kestlund joins17:12
* coblej leaves17:17
* dhlamb_ joins17:20
* th5 leaves17:41
* peichman leaves
* dhlamb_ leaves17:48
* whikloj leaves18:00
* kestlund leaves18:53
* dhlamb_ joins22:01
* dwilcox leaves22:14

Generated by Sualtam