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

Using timezone: Eastern Standard Time
* dwilcox joins07:59
* awoods joins08:26
* benpennell joins08:39
* whikloj joins09:08
* apb18 joins09:33
* yamil joins09:37
* peichman joins09:41
* github-ff joins09:50
[fcrepo4] whikloj opened pull request #1279: Add jbossts-properties.xml to fcrepo-webapp (master...fcrepo-2664-master) https://git.io/vNagi
* github-ff leaves
* github-ff joins09:51
[fcrepo4] whikloj closed pull request #1277: Add jbossts-properties.xml file (4.7-maintenance...fcrepo-2664) https://git.io/vNuFe
* github-ff leaves
* peichman leaves09:54
* peichman joins
* peichman leaves
* peichman joins09:58
* dhlamb joins10:06
<awoods>whikloj: I think we want a jbossts PR against the 4.7.5-RC branch, no?10:08
<whikloj>awoods: I don't know, I thought if the PR was being merged into RC-1 then any further RC would come from it10:16
<awoods>whikloj: yes... noting that RC-1 is a tag.10:17
the branch is 4.7.5-RC
<whikloj>awoods: so where did that PR merge into? Oh github you so crazy
<awoods>whikloj: I am not seeing a PR against anything "RC".
<whikloj>awoods: oh actually it was against 4.7.5-RC10:18
<awoods>whikloj: sorry. Your PR is there: https://github.com/fcrepo4/fcrepo4/commits/4.7.5-RC
<whikloj>https://github.com/fcrepo4/fcrepo4/pull/1278
right
<awoods>whikloj: it slipped by me
<whikloj>awoods: no worries, does that branch get pushed on top of 4.7-maintenance at the end?
<awoods>yes10:19
<whikloj>awoods++ # cool
<awoods>whikloj++ cool
<dbernstein>dbernstein: whikloj++ awoods++ cool.11:02
awoods, whikloj, et al: so with the fix in place, at what point do we want to cut RC-2?11:07
It seems to me that we should do it sooner rather than later since fcrepo4-vagrant is currently broken no?11:08
Unless you all see a compelling reason not to, I am happy to push new tags, artifacts, and community email this morning.11:10
<whikloj>dbernstein/awoods: I guess so, however I don't know that I want to force anyone to re-do their testing as this had a limited impact and the impact was that the whole application would not start
<awoods>dbernstein: RC-2++11:14
* bseeger joins11:16
<dbernstein>RC-2++11:17
Let’s stick with the process.11:18
<apb18>RC-2 testing makes sense. Sorry I wasn't really around Friday. Will do Windows and API-X.11:23
<dbernstein>apb18++
<bseeger>hey folks - I'm testing a PR that requires pushing in invalid (syntactically) content-type to fcrepo. The issue I'm running into is that it seems that tomcat/jetty (or jersey?) are catching the issue before it ever gets to the server itself.11:38
so I'm not sure how to test an invalid content-type. (I thought I had at one point, but I'm realizing that it wasn't the case).11:39
anyone have any clue how to do that or have thoughts on something I'm missing?
<dbernstein>bseeger: what is the error you’re seeing?11:40
<bseeger>dbernstein: I get a 400 with: The request sent by the client was syntactically incorrect.
dbernstein: my initial request: curl -X PUT --upload-file out.jpg -H "Content-Type: cannot open '/tmp/40e8273e-1ea8-48ec-916c-e78bc06ba1ea20170608-25523-m6g42x' (No such file or directory)" "http://localhost:8080/5.0-SNAP-FIX/rest/test/foo" -i -u fedoraAdmin:fedoraAdmin11:41
the content type is meant to be junk - somehow they were able to get that exact type in there in hyku11:42
<whikloj>bseeger: did you try a simpler bad content-type, like -H"Content-Type: bob/seeger"
<dbernstein>I think “bob/seeger” will succeed no?11:43
<bseeger>whikloj: :) yup - that's syntatically valid and the server is happy with it
<dbernstein>what happens if you try to remove the single quotes, bseeger?11:44
<bseeger>dbernstein: same deal - I think once it sees spaces it's considered invalid
<dbernstein>I’m thinking the quotes and possibly parens need to be encoded. What happens if you add %20 for the spaces?11:45
<whikloj>bseeger/dbernstein: Totally, what about -H"Content-type: bbbbbbbbbbbbbbbb"11:46
seems like according to section 5.1 (https://www.ietf.org/rfc/rfc2045.txt) you need a type + "/" + subtype to be syntatically valid11:47
<bseeger>replacing parens and spaces doesn't seem to work.
<whikloj>or even -H"Content-type: toast/jam" as toast is not a valid discreet type or composite type11:48
<dbernstein>bseeger: what is the verbose response from your original call?11:49
maybe drop it in a gist or something...11:50
<bseeger>dbernstein, whikloj: https://gist.github.com/bseeger/0022d54c2b8d444343d7f2192fae2b03
<whikloj>bseeger: Tomcat does not allow encoded quotations by default. We learned that with cantaloupe in CLAW-ville11:51
<bseeger>whikloj: good to know. I don't think I have any encoded quotes in there. I took the single quotes out. There are encoded spaces and parens11:52
* kefo joins11:57
<dbernstein>what happens when you take all the spaces, single quotes and parens?11:58
also something to try: remove the suffix on out.jpg and see how that changes things.12:00
<bseeger>stripping those out doesn't work. I have to take all that out and remove the extra '/' — then it meets the syntax
<kefo>Quick question: Should the tests reported on here be run against webapp-plus or the generic webapp?
https://wiki.duraspace.org/display/FF/Release+Testing+-+4.7.5#ReleaseTesting-4.7.5-DatabaseTests
<bseeger>so the thing is - I think tomcat is checking syntax — but how did hyku ever get that into fedora in the first place…? I'm wondering that out loud - I don't expect you all have all the answers.12:01
(It would be nice if you did though. ;)12:02
<dbernstein>right. Is it possible that no content was set in the hyku request and somehow jersey (or another software layer) attempted to resolve the content by inspecting the temp file on the server only to find that it wasn’t there for whatever reason?12:03
“no content” -> “no content type”12:04
* peichman leaves12:05
<apb18>kefo: webapp plus is ideal12:06
but it's sort of unspecified
bseeger, dbernstein: Is jetty any different with regard to content types?
(i.e. the one-click jar?
* peichman joins12:07
<bseeger>apb18 - I tried the one-click the other day with a similar result. I should try it again, though.
<kefo>Thanks @apb18. Actually, there's text above that suggests it should be webapp-plus, but it is a tad unclear. Anyways - and this question is more for my own enlightenment - what is the difference between webapp-plus and the generic webapp such that those particular tests are more beneficially run against webapp-plus (versus running the tests against both builds)?12:11
* peichman leaves12:12
* github-ff joins
[fcrepo4] dbernstein tagged fcrepo-4.7.5-RC-2 at 4.7.5-RC: https://git.io/vNaQM
* github-ff leaves
<bseeger>dbernstein, whikloj, apb18 - thanks for the thoughts, I'll work on this some more after lunch.12:13
* whikloj leaves12:16
<apb18>kefo: I don't think those particular tests have much specifically to do with webapp-plus. At the moment, the key distinguishing factor about webapp plus is WebAC. The tests don't specifically exercise that. So I see it as "make sure somebody verifies that fcrep-webapp-plus isn't obviously broken"
<dbernstein>awoods et al: release process question: yes or no: should I tag all submodules in the release with RC-2 even if there is no change to their respective 4.7.5-RC branches? I’m guessing “yes” for the sake of clarity.12:17
<kefo>apb18: Ahh… Thanks.
apb18: Well, I'm going to report on the generic webapp against a mysql backend because I'm halfway through those tests, but I will see about installing webapp-plus and testing that too, with both backends on linux.12:18
<apb18>kefo: excellent12:19
* whikloj joins12:27
* travis-ci joins12:29
fcrepo4/fcrepo4#5294 (fcrepo-4.7.5-RC-2 - 2fb8f5e : Jared Whiklo): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/fcrepo-4.7.5-RC-2
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/331910977
* travis-ci leaves
* dwilcox leaves12:32
* benpennell leaves12:42
* apb18_ joins12:47
* apb18 leaves12:48
* peichman joins
* dwilcox joins13:06
* benpennell joins13:30
* github-ff joins13:31
[fcrepo-webapp-plus] dbernstein tagged fcrepo-webapp-plus-4.7.5-RC-2 at 4.7-maintenance: https://git.io/vNVeG
* github-ff leaves
* github-ff joins13:36
[fcrepo-webapp-plus] dbernstein deleted fcrepo-webapp-plus-4.7.5-RC-2 at ba89ff9: https://git.io/vNVvf
* github-ff leaves
* github-ff joins13:37
[fcrepo-webapp-plus] dbernstein tagged fcrepo-webapp-plus-4.7.5-RC-2 at 4.7-maintenance: https://git.io/vNVeG
* github-ff leaves
* dwilcox leaves13:57
* dwilcox joins14:19
* travis-ci joins14:21
fcrepo4-exts/fcrepo-webapp-plus#237 (fcrepo-webapp-plus-4.7.5-RC-2 - ba89ff9 : Jared Whiklo): The build has errored.
Change view : https://github.com/fcrepo4-exts/fcrepo-webapp-plus/compare/fcrepo-webapp-plus-4.7.5-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-webapp-plus/builds/331941531
* travis-ci leaves
* travis-ci joins14:27
fcrepo4-exts/fcrepo-webapp-plus#238 (fcrepo-webapp-plus-4.7.5-RC-2 - ba89ff9 : Jared Whiklo): The build has errored.
Change view : https://github.com/fcrepo4-exts/fcrepo-webapp-plus/compare/fcrepo-webapp-plus-4.7.5-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-webapp-plus/builds/331944161
* travis-ci leaves
* travis-ci joins14:39
fcrepo4-exts/fcrepo-webapp-plus#237 (fcrepo-webapp-plus-4.7.5-RC-2 - ba89ff9 : Jared Whiklo): The build has errored.
Change view : https://github.com/fcrepo4-exts/fcrepo-webapp-plus/compare/fcrepo-webapp-plus-4.7.5-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-webapp-plus/builds/331941531
* travis-ci leaves
* github-ff joins14:55
[fcrepo4-vagrant] dbernstein created update-to-4.7.5-RC-2 (+1 new commit): https://git.io/vNVmK
fcrepo4-vagrant/update-to-4.7.5-RC-2 e86e264 Daniel Bernstein: Updates to 4.7.5 RC 2.
* github-ff leaves
* github-ff joins14:57
[fcrepo4-vagrant] dbernstein opened pull request #84: Updates to 4.7.5 RC 2. (4.7.5-RC...update-to-4.7.5-RC-2) https://git.io/vNVYn14:58
* github-ff leaves
* travis-ci joins15:02
fcrepo4-exts/fcrepo-webapp-plus#238 (fcrepo-webapp-plus-4.7.5-RC-2 - ba89ff9 : Jared Whiklo): The build has errored.
Change view : https://github.com/fcrepo4-exts/fcrepo-webapp-plus/compare/fcrepo-webapp-plus-4.7.5-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-webapp-plus/builds/331944161
* travis-ci leaves
* whikloj_ joins15:13
* whikloj leaves15:16
<dbernstein>If anyone would be willing to merge https://github.com/fcrepo4-exts/fcrepo4-vagrant/pull/84 it would help us keep the release process going.15:18
* github-ff joins15:19
[fcrepo-module-auth-rbacl] dbernstein tagged fcrepo-module-auth-rbacl-4.7.5-RC-2 at 4.7-maintenance: https://git.io/vNVss
* github-ff leaves
* github-ff joins15:20
[fcrepo-mint] dbernstein tagged fcrepo-mint-4.7.5-RC-2 at 4.7-maintenance: https://git.io/vNVs8
* github-ff leaves
* github-ff joins15:22
[fcrepo-module-auth-webac] dbernstein tagged fcrepo-module-auth-webac-4.7.5-RC-2 at 4.7-maintenance: https://git.io/vNVsg
* github-ff leaves
* travis-ci joins15:23
fcrepo4/fcrepo-module-auth-webac#220 (fcrepo-module-auth-webac-4.7.5-RC-2 - 0c18396 : Jared Whiklo): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo-module-auth-webac/compare/fcrepo-module-auth-webac-4.7.5-RC-2
Build details : https://travis-ci.org/fcrepo4/fcrepo-module-auth-webac/builds/331985666
* travis-ci leaves
* github-ff joins15:24
[fcrepo-audit] dbernstein tagged fcrepo-audit-4.7.5-RC-2 at 4.7-maintenance: https://git.io/vNVsS
* github-ff leaves
* travis-ci joins15:26
fcrepo4-exts/fcrepo-mint#81 (fcrepo-mint-4.7.5-RC-2 - b8ccfc0 : Jared Whiklo): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-mint/compare/fcrepo-mint-4.7.5-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-mint/builds/331985308
* travis-ci leaves
* travis-ci joins15:31
fcrepo4-exts/fcrepo-module-auth-rbacl#154 (fcrepo-module-auth-rbacl-4.7.5-RC-2 - abfae37 : Jared Whiklo): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-module-auth-rbacl/compare/fcrepo-module-auth-rbacl-4.7.5-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-module-auth-rbacl/builds/331984902
* travis-ci leaves
* travis-ci joins15:33
fcrepo4-exts/fcrepo-audit#186 (fcrepo-audit-4.7.5-RC-2 - dd7a9a5 : dbernstein): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-audit/compare/fcrepo-audit-4.7.5-RC-2
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-audit/builds/331986387
* travis-ci leaves
* apb18_ leaves15:57
* whikloj_ leaves16:04
* whikloj joins
* whikloj leaves16:07
* whikloj_ joins
<bseeger>hi folks, how stable is 5.0 snapshot? Locally I'm seeing all the webac recipe tests fail.16:11
not sure if it's something I did (I do have a small change in another module), or if it's a result of other separate changes.
failures look like this: WebACRecipesIT.testRegisterNamespace:696->ingestObj:119 expected:<201> but was:<409>16:12
WebACRecipesIT.testRegisterNodeType:713->ingestObj:119 expected:<201> but was:<409>
so I'm suspect that it's part of the spec changes that are being made…
<whikloj_>bseeger: not sure, anything in the logs?
<bseeger>the failsafe logs?16:13
failsafe report logs, that is
<whikloj>bseeger: oh these are tests...umm16:14
bseeger: So the fcrepo-module-auth-webac is failing?
<bseeger>whikloj: locally for me.16:15
whikloj: wouldn't that be reflected on github as well?
(if it weren't a local problem)
<whikloj>bseeger: ok, give me a couple and I'll build it now.
bseeger: it should yes16:16
<bseeger>whikloj: thanks!
whikloj: I can also create a clean repo - in case you're busy
<whikloj>bseeger: I'm always super never busy
or never super busy or busy super never16:17
<bseeger>I think it's me. :( let me know if you see that as well.16:24
whikloj: but you're always super awesome!16:25
<whikloj>bseeger: did you build fcrepo-module-auth-rbacl too? I think it needs to be still16:26
<bseeger>no….hmmm16:27
<whikloj>I could be wrong
bseeger: build success16:31
<bseeger>whikloj: okay - thanks for checking. Even my minor change must have missed a case.16:32
it's me, not you, fcrepo
<whikloj>haha
bseeger: oh wait I built 4.7.5 not master
bseeger: but still BUILD SUCCESS16:34
<bseeger>whikloj: :) I still suspect it's my code.16:35
<whikloj>bseeger: I'll take your word for it.16:56
<bseeger>thanks for your help, though. Have a nice night!16:57
* bseeger leaves
* travis-ci joins17:06
fcrepo4-labs/Fedora-API-Test-Suite#65 (master - 01ccd6d : ibr-jabrego): The build passed.
Change view : https://github.com/fcrepo4-labs/Fedora-API-Test-Suite/compare/98a54b301e37...01ccd6d444ef
Build details : https://travis-ci.org/fcrepo4-labs/Fedora-API-Test-Suite/builds/332029150
* travis-ci leaves
* travis-ci joins17:08
fcrepo4-labs/Fedora-API-Test-Suite#66 (master - eb18d59 : ibr-jabrego): The build passed.
Change view : https://github.com/fcrepo4-labs/Fedora-API-Test-Suite/compare/01ccd6d444ef...eb18d5973712
Build details : https://travis-ci.org/fcrepo4-labs/Fedora-API-Test-Suite/builds/332029730
* travis-ci leaves
* travis-ci joins17:09
fcrepo4-labs/Fedora-API-Test-Suite#67 (master - 924ab26 : ibr-jabrego): The build passed.
Change view : https://github.com/fcrepo4-labs/Fedora-API-Test-Suite/compare/eb18d5973712...924ab2664fe2
Build details : https://travis-ci.org/fcrepo4-labs/Fedora-API-Test-Suite/builds/332030548
* travis-ci leaves
* travis-ci joins17:11
fcrepo4-labs/Fedora-API-Test-Suite#68 (master - 4166b4d : ibr-jabrego): The build passed.
Change view : https://github.com/fcrepo4-labs/Fedora-API-Test-Suite/compare/924ab2664fe2...4166b4db27b1
Build details : https://travis-ci.org/fcrepo4-labs/Fedora-API-Test-Suite/builds/332031374
* travis-ci leaves
* peichman leaves17:15
* yamil leaves17:35
* apb18_ joins17:48
* dhlamb leaves17:52
* benpennell leaves17:54
* whikloj leaves18:00
* kefo leaves18:24
* benpennell joins19:42
* apb18_ leaves19:58
* benpennell leaves20:36
* dwilcox leaves21:30
* benpennell joins22:17
* benpennell1 joins22:18
* benpennell leaves
* benpennell1 leaves22:20
* benpennell joins
* awoods leaves23:17
* benpennell leaves23:28

Generated by Sualtam