IRC Archive for channel #xwiki

Last modified by Vincent Massol on 2012/10/18 19:12

sdumitriu left at 01:34 (Quit: Leaving.
Enygma` left at 01:40 (Ping timeout: 276 seconds
abusenius left at 01:53 (Ping timeout: 240 seconds
mflorea joined #xwiki at 07:43
mariusbutuc joined #xwiki at 07:52
sburjan joined #xwiki at 08:24
florinciu joined #xwiki at 08:27
vmassol joined #xwiki at 08:39
Enygma` joined #xwiki at 09:19
evalica joined #xwiki at 09:20
sburjan - (09:29): http://code.xwiki.org/xwiki/bin/view/Applications/AnnotationsApplication
sburjan - (09:29): please check the download link for this app
sburjan - (09:29): is it correct ?
sburjan - (09:29): shouldnt point to a xar file ?
florinciu left at 09:31 (Read error: Connection reset by peer
lucaa left at 09:38 (Ping timeout: 240 seconds
vmassol - (09:46): not anymore, thomas should have sent a mail on the dev list to explain it....
vmassol - (09:46): I'll do it
arkub joined #xwiki at 09:49
vmassol - (09:50): sburjan: mail sent
jvdrean joined #xwiki at 09:51
vmassol left at 09:52 (Quit: Leaving.
KermitTheFragger joined #xwiki at 09:53
vmassol joined #xwiki at 09:54
vmassol1 joined #xwiki at 09:56
vmassol left at 09:56 (Read error: Connection reset by peer
lucaa joined #xwiki at 10:09
sburjan - (10:12): wait.. what are you talking about .. about the Annotations ?
sburjan - (10:13): oh
sburjan - (10:13): i read the mail
abusenius joined #xwiki at 10:13
sburjan - (10:13): but isn't this a bit hard for the end-user ?
mariusbutuc - (10:19): sburjan: sounds good to me, and a lot easier from the dev pov
mflorea left at 10:19 (Quit: Leaving.
sburjan - (10:20): for devs yes, but for normal users that want to try it out,etc
sburjan - (10:20): it should be specified that the "installer" is actually the xar file
sburjan - (10:20): so that end users know too what to do
lucaa - (10:22): actually I think it's a very good idea to link to a place with all versions for download
lucaa - (10:22): users will know they need to pick the latest version
lucaa - (10:22): and it's a lot easier for development, to make sure that latest version is always available
lucaa - (10:23): however, this will not be possible for all artifacts, since not all are released and can be linked to a folder in maven
tmortagne joined #xwiki at 10:26
lucaa - (10:26): the only thing would be indeed kind of making it easy for the users to know that it's the .xar / .jar etc. that they want from the list (and not something else). Most of the users on code.xwiki.org will know it, I think, because that's why they are there but we can never be sure
florinciu joined #xwiki at 10:36
florinciu left at 10:36 (Client Quit
sburjan - (10:39): what server is at myxwiki.org .. it always have problems and gets restarted
florinciu joined #xwiki at 10:53
sdumitriu joined #xwiki at 11:07
mflorea joined #xwiki at 11:10
vmassol joined #xwiki at 11:13
nuvolari left at 11:14 (Quit: No Ping reply in 180 seconds.
nuvolari joined #xwiki at 11:14
vmassol1 left at 11:15 (Ping timeout: 245 seconds
florinciu left at 11:22 (Quit: Leaving.
nuvolari left at 11:27 (Quit: http://quassel-irc.org - Chat comfortably. Anywhere.
evalica - (11:28): hi sburjan I'm testing 2.4 M2 - and annotations are working only if in Admin- Presentation, the Skin is set with value XWiki.DefaultSkin; otherwise (custom, colibri, toucan) annotations are not working - can you test this? do you know something about this?
sburjan - (11:28): let me check
sburjan - (11:30): 2.4 M2 or latest snapshot ?
evalica - (11:30): 2.4 M2
sdumitriu - (11:31): lucaa: This is for you, read above
sdumitriu - (11:31): If anybody else wants to study the memory dump on myxwiki, http://incubator.myxwiki.org/xwiki/snaps/Bootstrap-2010-07-06-periodic-7.snapshot (800M file)
sburjan - (11:32): mine annotations work in toucan too
sburjan - (11:32): let me check albatros
evalica - (11:33): what value have you put to set the skin to toucan?
sburjan - (11:33): toucan
sburjan - (11:33): in the Skin input field
evalica - (11:34): I tested twice - first I set a custom skin, ex. Skin.NewSkin ... and after toucan, colibri, etc are not working for me anymore
sburjan - (11:34): albatross works too
sburjan - (11:34): well.. I tested for default, then changed to toucan, then to albatross
sburjan - (11:34): let me check on other pages, i only tested on Main
evalica - (11:34): sburjan: I'll test again after the meeting - thanks
evalica - (11:34): The presentation is for Wiki level
sdumitriu - (11:34): Size increase over time:
sdumitriu - (11:34): 23:30 - 440M
sdumitriu - (11:34): 00:30 - 555M
sdumitriu - (11:34): 01:30 - 660M
sdumitriu - (11:34): 02:30 - 820M
sdumitriu - (11:34): 03:10 - crash
sdumitriu - (11:34): 04:10 - 325M
sdumitriu - (11:34): 05:10 - 475M
sdumitriu - (11:34): 06:10 - 610M
sdumitriu - (11:34): 07:10 - 700M
sdumitriu - (11:34): 08:10 - 800M
lucaa - (11:34): evalica: the annotations don't work on other skin but collibri
sdumitriu - (11:34): 08:20 - crash
lucaa - (11:34): I mean the UI
evalica - (11:35): so If I want to extend them I should rewrite the annotations - k
sburjan - (11:35): that's odd.. mine works
evalica - (11:35): lucaa: thanks - but this should be fixed
lucaa - (11:35): evalica: extend?
evalica - (11:35): I wanted to extend Colibri with a custom skin .. and is not working
evalica - (11:36): custom skin that has as base colibri
sburjan - (11:36): I don;t have other skins .. i only have the 3 bundled ones
lucaa - (11:36): evalica: well, as long as you don't modify much, it should work
evalica - (11:36): lucaa: I've overriden commentsinline.vm stopped working
evalica - (11:36): what means don't modify much?
lucaa - (11:37): commentsinline should not make it fail
lucaa - (11:37): wdym it does not work? no UI or partial UI or UI but crashing?
evalica - (11:37): so .. they work ... add / visualizing ... the tab from docextra is not working
lucaa - (11:37): ok
lucaa - (11:37): ok, I was about to tell you about this
lucaa - (11:38): the tab from docextra is rendered using a template attached to XWiki.DefaultSkin
lucaa - (11:38): if you want to extend, you should also get it the template from there and attach it to your skin
lucaa - (11:38): and it will work
lucaa - (11:38): agreeded that it should be less complex than that
evalica - (11:38): as a hack - but this shouldn't be the normal behavior
lucaa - (11:38): s/agreeded/agreed
evalica - (11:39): k thanks lucaa - this would solve my problem
sburjan - (11:40): I'm still wondering how annotations work even when I use toucan
evalica - (11:40): sburjan sorry I wasn't more specific .. I meant the #annotation tab from docextra
lucaa - (11:40): evalica: yes, but that's a little more complicated. I avoided bundling the annos in the war, as in add jss in there, add templates, etc, I prefered to keep it in .xar form, and I still prefer.
sburjan - (11:40): oh, I see
vmassol1 joined #xwiki at 11:43
vmassol left at 11:43 (Read error: Connection reset by peer
lpereira joined #xwiki at 11:44
vmassol joined #xwiki at 11:46
vmassol1 left at 11:46 (Read error: Connection reset by peer
evalica - (11:47): lucaa: also why didn't you add itemCount of existing annotations for Annotation tab, like Annotations (3) ? same reason?
lucaa - (11:48): no, because that would require an extra ajax call, which would be time costly, but it's possible to do it without annoying the user
lucaa - (11:48): there should be an issue reported for it in the XAANNOTATIONS project, vmassol should have done it iirc
evalica - (11:48): from consistency reason - that should be added - also in the shortcut area up
sdumitriu - (11:49): vmassol: VelocimacroManager takes up 400M out of 800M
evalica - (11:53): lucaa: yes found it thanks
vmassol - (11:54): otp (meeting)
evalica - (11:56): sdumitriu: about Class editor - when you import a skin from somewhere else (where you had a different Skins Class declaration) , the file shows only the fields from the current Skins Class (even thou those fields are in the description and active) - is there a way to show them somehow?
evalica - (11:58): something like "The following properties where deleted from the class  XWiki.XWikiSkins and are now deprecated:" but more like "This object also contains other properties, that are not default in the Class" ... or something
abusenius - (12:03): sdumitriu, which JRE is used on myxwiki?
abusenius - (12:04): I have problems runnign xwiki on sun 1.6.0.20 x86_64, it leaks memory (sun 1.5.0.22 works fine)
nuvolari joined #xwiki at 12:05
nuvolari left at 12:05 (Changing host
nuvolari joined #xwiki at 12:05
tmortagne - (12:16): myxwiki.org is using Java 1.6 AFAIK but i don't know which exact version
florinciu joined #xwiki at 12:25
lucaa left at 12:29 (Ping timeout: 265 seconds
mariusbutuc left at 12:48 (Ping timeout: 252 seconds
arkub left at 12:55 (Quit: Leaving
sdumitriu - (13:15): abusenius:
sdumitriu - (13:15): java version "1.6.0_12"
sdumitriu - (13:15): Java(TM) SE Runtime Environment (build 1.6.0_12-b04)
sdumitriu - (13:15): Java HotSpot(TM) 64-Bit Server VM (build 11.2-b01, mixed mode)
lucaa joined #xwiki at 13:18
SvenDowideit left at 13:56 (Read error: Operation timed out
vmassol - (14:08): sdumitriu: sorry was in meeting the whole morning. Any disocvery re the memory issue?
sdumitriu - (14:09): Yep
sdumitriu - (14:09): Since macros don't override each other, now each wiki has its own set of macros
sdumitriu - (14:09): Macro overload
sdumitriu - (14:09): The only solution I can think of is to use a LRU to store macros
vmassol - (14:10): yes but once the page has finished rendernig it's cleared
sdumitriu - (14:10): No it's not
sdumitriu - (14:10): That's the problem
sdumitriu - (14:10): Macros are kept in a cache forever
vmassol - (14:10): the macros for the page namespace that is
vmassol - (14:10): I'm clearing them
sdumitriu - (14:10): For a page, maybe
vmassol - (14:10): that was the purpose of my fix
vmassol - (14:10): I only changed velcotiy macros for pages
sdumitriu - (14:10): Hm..
vmassol - (14:11): not velocity macros in templates
vmassol - (14:11): these are still registered with a common namespace
sdumitriu - (14:11): Are you sure?
sdumitriu - (14:11): Well, something's not working
vmassol - (14:12): here's what I call:
vmassol - (14:12):                     Utils.getComponent(VelocityManager.class).getVelocityEngine().clearMacroNamespace(documentName);
vmassol - (14:12): which internally calls:
vmassol - (14:12):         this.rsvc.dumpVMNamespace(templateName);
vmassol - (14:13): which calls:
vmassol - (14:13):         return vmManager.dumpNamespace(namespace);
sdumitriu - (14:13): Hm...
vmassol - (14:13): (in VelocimacroManager)
sdumitriu - (14:13): I don't think macros are namespaced in that cache
sdumitriu - (14:13): I'll investigate more the structure of that cache
vmassol - (14:13): they are
vmassol - (14:14): look at VelocimacroManager sources
vmassol - (14:14): (I have it under my eyes)
vmassol - (14:14):     /** Hash of namespace hashes. */
vmassol - (14:14):     private final Map namespaceHash = MapFactory.create(17, 0.5f, 20, false);
vmassol - (14:15): let me put a breakpoint just to test locally
SvenDowideit joined #xwiki at 14:15
sdumitriu - (14:15): OK, they are namespaced
sdumitriu - (14:15): But they are not cleared
sdumitriu - (14:16): Look in o.a.v.runtime.VelocimacroManager
vmassol - (14:16): I'm there
vmassol - (14:16):                 Map h = (Map) namespaceHash.remove(namespace);
sdumitriu - (14:16): btw, why are there 2 instances of org.xwiki.velocity.internal.DefaultVelocityEngine?
vmassol - (14:17): there's only one when you run locally but there are as many as you have skins
vmassol - (14:17): jconsole will give you info on that btw
sdumitriu - (14:17): Huh
sdumitriu - (14:18): K, something strange
sdumitriu - (14:18): There are 2 VelocimacroManagers
vmassol - (14:18): I can see the remove called in debug mode
sdumitriu - (14:18): One is linked from DefaultVelocityEngine, one is root
sdumitriu - (14:19): The one linked from our code is almost empty
sdumitriu - (14:19): The other one is the one that's full
arkub joined #xwiki at 14:26
sburjan - (14:27): does anyone know where could I find some JUnit tests on the SVN so I can take a look at them ?
sburjan - (14:27): something more larger that 2-3 tests
vmassol - (14:28): sburjan: look for all classes implementing AbstracMockingComponentTestCase for ex
vmassol - (14:28): this is the new method (cf mail I sent over the weekend about it)
sburjan - (14:28): platform or enterprise ?
vmassol - (14:28): platform
sburjan - (14:28): ok
fmancinelli joined #xwiki at 14:29
vmassol - (14:32): fmancinelli: the build is failing on hudson in core
vmassol - (14:32): http://hudson.xwiki.org/job/xwiki-platform-core/6229/org.xwiki.platform$xwiki-rest-model/console
vmassol - (14:34): sdumitriu: let me know if you need my help to debug the velocitymacromanager thing. I'm working on something else right now but I can help if need be.
fmancinelli - (14:34): vmassol: Yes, fixing it
arkub left at 14:37 (Ping timeout: 260 seconds
sburjan - (14:39): vmassol,  : I can't find the shared tests
tmortagne - (14:41): sburjan: http://svn.xwiki.org/svnroot/xwiki/platform/core/trunk/xwiki-shared-tests/
tmortagne - (14:41): that's the tools
tmortagne - (14:41): you can find example in most of the core project
tmortagne - (14:41): xwiki-core have a lot of them
tmortagne - (14:42): but it also have a lot of bad examples :)
DSpair joined #xwiki at 14:42
DSpair - (14:42): Good morning all..
tmortagne - (14:42): sburjan: see http://dev.xwiki.org/xwiki/bin/view/Community/Testing
vmassol - (14:43): english guys would you use ""Looking for professional services? Go on XWiki.com" or ""Looking for professional services? Go to XWiki.com" ?
DSpair - (14:43): Brit english, or US?
tmortagne - (14:44): US probably
DSpair - (14:44): "Go to" would be the common way in US English.
tmortagne - (14:45): vmassol: GG the advertisement on IRC chat ;)
DSpair - (14:45): vmassol: You could avoid US/UK distinction with the phrase "Navigate to XWiki.com".
DSpair - (14:46): Or perhaps "Link to XWiki.com"
jvdrean left at 14:46 (Quit: Leaving.
jvdrean joined #xwiki at 14:47
vmassol - (14:49): tmortagne: that wasn't my purpose!
vmassol - (14:50): I also think Go to is better
vmassol - (14:50): FYI guys I'm going to add an image advertisement on the download page
vmassol - (14:50): according to our governance rule
vmassol - (14:51): http://dev.xwiki.org/xwiki/bin/view/Community/Governance
DSpair - (14:53): Question on XAR files.
sburjan - (14:53): thanks vmassol , tmortagne
DSpair - (14:54): The exported XML file for a class has a UUID... How broken would it be to have a duplicate UUID?
DSpair - (14:54): The reason I ask is that I would like to programmatically (from a script) generate a bunch of new object instances and use a default template.
vmassol - (14:57): evalica: could you help me on styling http://www.xwiki.org/xwiki/bin/view/Main/Support correctly?
vmassol - (14:57): (so that there's no scroll and so that the text wraps correctly :))
tmortagne - (14:58): DSpair: there UUIS only for object, i doubt you have a UUID in the XML when exporting just a class
tmortagne - (14:58): s/UUIS/UUID/
tmortagne - (14:59): DSpair: if the question is "how do i generate UUID" the answer is use the proper API and it will be automatically generated for you ;)
fmancinelli - (15:09): vmassol: Should be fixed now. I was compiling with JDK1.6 which happens to have some previously required dependencies (JAXB) in the standard libraries.
mariusbutuc joined #xwiki at 15:10
arkub joined #xwiki at 15:20
DSpair - (15:23): tmortagne, You are correct, I meant to say object instance/page.
DSpair - (15:24): tmortagne, As for using the proper APIs, that's not really a very good option as I want to convert the information from an incompatible data source.
tmortagne - (15:24): DSpair: you are working in pure XML you mean ?
tmortagne - (15:24): no in the wiki ?
DSpair - (15:25): tmortagne, I have an existing external database which I will write a script to generate XML files from that database.
tmortagne - (15:25): the best is probably to simply not put any UUID in the XML
tmortagne - (15:25): a unique one will be automatically generated
DSpair - (15:25): tmortagne, That's exactly what I was hoping!
DSpair - (15:25): Thanks so much!
tmortagne - (15:25): in fact IMO we should probably not have UUID in the exported XML
DSpair - (15:26): Because it could conflict in another XWiki instance...
tmortagne - (15:26): yep
tmortagne - (15:26): or at least not take then into account if we don't import as backup pack
DSpair - (15:26): Would you like me to put in an RFE?
tmortagne - (15:26): i will check that
tmortagne - (15:26): (it's possible we already do that i don't know)
tmortagne - (15:27): but i doubt that
tmortagne - (15:27): DSpair: RFE ?
DSpair - (15:28): RFE = Request For Enhancement
tmortagne - (15:28): DSpair: ho ok sorry :) Yep sure do it
sburjan - (15:47): vmassol,  : so AbstractMockingComponentTestCase should be used as a ProtoFramework for testing with JUnit and JMock ?
vmassol - (15:48): sburjan: there are some info here too: http://dev.xwiki.org/xwiki/bin/view/Community/Testing
vmassol - (15:48): sburjan: yes
sburjan - (15:48): read them :P
sburjan - (15:49): so every JUnit test will have to be "converted" to use this class ?
arkub left at 15:50 (Ping timeout: 260 seconds
mariusbutuc left at 16:23 (Ping timeout: 245 seconds
vmassol - (16:33): sburjan: new tests should be written this way yes
vmassol - (16:33): and indeed old tests should be converted slowly to it
sdumitriu - (16:34): vmassol: Found two possible leaks
vmassol - (16:34): ahah
sdumitriu - (16:34): 1: exceptions
sdumitriu - (16:34): 2: parsed SX
vmassol - (16:35): tmortagne: we're supposed to stabilize XE…. :)
vmassol - (16:35): (upgrading versions is a risk)
sdumitriu - (16:36): Ah, no
sdumitriu - (16:36): Forget 1
sdumitriu - (16:36): That's minor
sdumitriu - (16:36): There are two getRenderedContent methods in XWikiDocument
sdumitriu - (16:36): One has the clear, the other doesn't
tmortagne - (16:36): vmassol: upgrading to bugfixing releases
sdumitriu - (16:36): And the other one is called for rendering panels
sdumitriu - (16:37): So each page with panels leaks macros
sdumitriu - (16:37): Which means all pages
sdumitriu - (16:37): vmassol: Do you want to fix it, or can I go on?
vmassol - (16:38): sdumitriu: hmmm
vmassol - (16:38): I discussed with tmortagne about this
vmassol - (16:38): maybe we missed something
vmassol - (16:38): we had voluntarily done the clear only in one method
tmortagne - (16:38): vmassol: the cleanup was not supposed to be done in getRenderedContent
tmortagne - (16:39): but at the end of the whole page rendering process
tmortagne - (16:39): when we talked about it
vmassol - (16:39): tmortagne: you're changing the history :)
tmortagne - (16:39): no
tmortagne - (16:39): i don't
vmassol - (16:39): since we did it together
vmassol - (16:39): but that's ok
sdumitriu - (16:39): That might still leak, not all documents are rendered in the current document
tmortagne - (16:39): i'm sure of what i'm saying
tmortagne - (16:40): it's useless to cleanup only for the rendered content part
sdumitriu - (16:40): You can do $someOtherDoc.getRenderedContent
vmassol - (16:40): tmortagne: you don't remember that we agreed that checking isInRenderingEngine was the way to check inside getRendereContent that the rendering was over?
tmortagne - (16:41): yep we talked about that but we also talked about panels if i remember well
vmassol - (16:42): anyway the question is: how do we fix it?
tmortagne - (16:44): vmassol: didn't you changed something in the way panels was rendered to make then use the same entry point ?
vmassol - (16:44): no
vmassol - (16:44): I haven't changed anythin
arkub joined #xwiki at 16:44
tmortagne - (16:44): i think we talked about that before saying that we had a single entry point to be able to cleanup
vmassol - (16:45): maybe I didn't really understand that part
vmassol - (16:45): and thus didn't change anythng
vmassol - (16:45): what's strange though is that in my tests using jconsole I didn't see the cache growing
vmassol - (16:45): (velocity macro cache)
tmortagne - (16:45): (that's what even lead use to talk about future design)
tmortagne - (16:45): s/use/us/
vmassol left at 16:52 (Ping timeout: 245 seconds
vmassol joined #xwiki at 16:58
DSpair - (17:01): OK, clean install of XWiki, new database (postgres 8.3), and imported the XWiki default xar.
DSpair - (17:01): But I still cannot use attached JAR files with my Groovy applications.
lpereira left at 17:02 (Quit: Leaving.
vmassol - (17:02): tmortagne: ?
vmassol - (17:02): sdumitriu: I think it should be ok to add the same check
tmortagne - (17:02): vmassol: and don't forget to put the cleanup in a finally like sdumitriu suggested
vmassol - (17:03): sdumitriu: can you make the change or would you prefer thomas or me do it?
sburjan left at 17:06 (Quit: Ex-Chat
vmassol - (17:07): seems like all core is broken on hudson
tmortagne - (17:13): vmassol: i'm on it
tmortagne - (17:13): small mistake i did
vmassol - (17:13): it = hudson or the memory leak?
vmassol - (17:13): ok hudson
tmortagne - (17:13): hudson
sdumitriu - (17:20): I'll take the leak
vmassol - (17:23): ok thanks
sdumitriu - (17:29): vmassol, tmortagne: In the future we should refactor the way we use velocity macros
sdumitriu - (17:29): Since the current way is very inefficient
sdumitriu - (17:29): We shouldn't use {{include}} to import macros, but have a special {{useMacroLibrary}} or something like that
vmassol - (17:29): oh definitely
vmassol - (17:29): we need
sdumitriu - (17:29): Which only parses once the document and reuses the same instance each time
vmassol - (17:29): the same mechanism as for wiki macros
vmassol - (17:29): but for velocity macros
vmassol - (17:29): ie objects
vmassol - (17:30): and a manager that listens
sdumitriu - (17:30): Yep
sdumitriu - (17:30): Maybe for 2.5
vmassol - (17:30): maybe
sdumitriu - (17:30): I'd like 2.5 to be targeted on security and performance
tmortagne - (17:30): sdumitriu: sure, but not sure for 2.5
vmassol - (17:31): I'll send a mail for polling what people would like to see in 2.5 in brainstorming mode
tmortagne - (17:31): (but i would love to make 2.5 target on security/performance/bugfixes too)
sdumitriu - (17:33): 2.5 should be around the start of november
sdumitriu - (17:34): Going back to the leak
sdumitriu - (17:34): I'll just clean the cache in the other getRenderedContent method for the moment
sdumitriu - (17:35): If !isInRenderingEngine
sdumitriu - (17:35): Is that OK?
tmortagne - (17:35): sdumitriu: yep and also put all that in the finally (same for the one that already have the clean)
vmassol - (17:35): sdumitriu: sounds ok to me, please also fix the finally
sdumitriu - (17:36): Sure
tmortagne left at 17:38 (Quit: Leaving.
lpereira joined #xwiki at 18:13
evalica - (18:34): q: if I have a macro how can I found out in which file it is defined?
florinciu left at 18:35 (Quit: Leaving.
fmancinelli left at 18:39 (Ping timeout: 260 seconds
mflorea left at 18:45 (Quit: Leaving.
abusenius left at 18:52 (Ping timeout: 240 seconds
sdumitriu - (19:06): evalica: Search
arkub left at 19:09 (Quit: Leaving
lucaa - (19:12): evalica: since some recent version there are some properties of the macro descriptors to get this info. Basically you need to write a bit of code to get this descriptor to get the page name. I can search for the specific way to do it, if it would suit your purpose
lucaa left at 19:16 (Quit: Leaving.
evalica - (19:16): thanks
evalica left #xwiki at 19:18
abusenius joined #xwiki at 19:19
KermitTheFragger left at 19:23 (Quit: Leaving
CalebJamesDeLisl - (19:25): abusenius: I got crypto.signText to work. What I had to do was create a CA cert on the server and sign the client cert, then the client has to install the ca cert (which is surprisingly easy).
abusenius - (19:26): cool
sdumitriu - (19:26): Cool
sdumitriu - (19:27): abusenius: What's the status on CSRF prevention?
abusenius - (19:27): sdumitriu: the component that does it is small and working
abusenius - (19:28): most of the important applications, all templates and all actions are patched (up to possible bugs)
abusenius - (19:29): most problematic are the tests (especially selenium1)
florinciu joined #xwiki at 19:29
abusenius - (19:29): they use url to delete/create/etc pages all over the place
abusenius - (19:30): it would be cool if a couple of people would test it
abusenius - (19:32): btw, CalebJamesDeLisl, I'm writing a prototype of the signed macro and a crypto component
sdumitriu - (19:32): I think it should be applied early on 2.5
abusenius - (19:33): +1 :)
CalebJamesDeLisl - (19:33): I'm the worst for doing that (save with url). You can't post and it takes forever to navigate around to the page you want.
sdumitriu - (19:33): abusenius: Do you think the code (except tests) is ready?
sdumitriu - (19:33): CalebJamesDeLisl: +1, me too
abusenius - (19:33): I think yes
sdumitriu - (19:34): Initially I was against fixing CSRF because of this (I know, I'm lazy)
abusenius - (19:34): im principle it is ok to save with url, as long as the token is there
abusenius - (19:34): we just need to read it out of a page and use everywhere
sdumitriu - (19:35): Ah, what about REST?
DSpair - (19:35): CalebJamesDeLisl, I'm back to debugging the attached jars again. Has anyone had a chance to investigate?
abusenius - (19:35): it is not protected
sdumitriu - (19:35): Is it still open to CSRF?
abusenius - (19:35): yes
CalebJamesDeLisl - (19:35): We can turn it off for tests for the beginning. I'm +1 to having some protection rather than waiting until everything is perfect before installing.
sdumitriu - (19:35): K
abusenius - (19:35): it is easy to fix though, I was talking to Fabio about it
sdumitriu - (19:36): Sure it's easy, it just has to be done in a convenient way
abusenius - (19:36): and we need to fix applications that use it
CalebJamesDeLisl - (19:36): DSpair: I haven't had any chance, I noticed you sent mail, Thomas who responded knows a lot more about components and rendering than me.
abusenius - (19:37): if somebody wants to test CSRF protection, ping me, I'll update the patches
DSpair - (19:37): CalebJamesDeLisl, OK, thanks. I'll wait for him to respond further.
CalebJamesDeLisl - (19:53): abusenius: signText uses pkcs7 so if you're working on a validator, that's probably the best choice for a standard. I can send you my (still hackish) work to date so you can play with it.
florinciu left at 19:56 (Quit: Leaving.
CalebJamesDeLisl - (20:04): DSpair: I'll try duplicating your problem in 2.3.0, I'm afraid the answer is wither going to be "there was a bug, upgrade" or it's something about your jvm/operating system/etc.
DSpair - (20:16): CalebJamesDeLisl, I've tried on Linux and Windows, Java 5 and 6, new and old installation of 2.3.1...
CalebJamesDeLisl - (20:17): Have you tried it on installations other than 2.3.1?
CalebJamesDeLisl - (20:17): *versions
DSpair - (20:17): CalebJamesDeLisl, No.
CalebJamesDeLisl - (20:22): I have 2.3.0 sitting on my computer, I can give that a try.
DSpair - (20:26): CalebJamesDeLisl, Thanks. I would appreciate that.
sdumitriu - (20:37): I'll put non-leaky versions of the core and of the skinx on myxwiki to see how it behaves
sdumitriu - (20:38): If it's OK, we can release RC1 tomorrow
DSpair left at 20:56 (Quit: Leaving
sdumitriu - (21:10): K, again "inexisting" documents hidden behind the cache
sdumitriu - (21:10): Something else is wrong on myxwiki
sdumitriu - (21:15): vmassol: Ping
DSpair joined #xwiki at 21:33
CalebJamesDeLisl - (21:41): DSpair: Can't repeat in 2.3.0. Are you using jetty?
DSpair - (21:41): No, using PostgreSQL
CalebJamesDeLisl - (21:41): jetty vs. tomcat.
DSpair - (21:41): Tomcat
DSpair - (21:42): Sorry, I was thinking hsqldb
CalebJamesDeLisl - (21:42): Maybe that has something to do with it. I'm running jetty as the back end.
DSpair - (21:42): Let me try 2.3.0 in tomcat  and see
DSpair - (21:54): CalebJamesDeLisl:  2.3.0 fails inside of Tomcat as well.
DSpair - (21:54): I guess it could be something to do with the container properties, but I am running with the security manager turned off.
CalebJamesDeLisl - (21:57): I think there must be something going on with the Tomcat loader. I'm out of my league with tomcat. Maybe try asking the tomcat people about the error message you get? I seem to recall there were some google hits for that message. Good luck.
sdumitriu - (21:58): But it works on xwiki.org
sdumitriu - (21:58): Which is using Tomcat
DSpair - (21:58): CalebJamesDeLisl:  On the attached jars, I don't get ANY message. That thread just stops.
sdumitriu - (21:59): Are you sure it's not something wrong with the jar itself?
DSpair - (21:59): CalebJamesDeLisl:  Even when I bump logging up to debug, I get nothing.
DSpair - (21:59): sdumitriu:  It doesn't matter what Jar I use.
DSpair - (22:00): I could create a blank JAR file using "jar -cvf myJar.jar " and it would still cause the error.
sdumitriu - (22:00): OK, I'll try on myxwiki a live example
sdumitriu - (22:01): Simple hello world from groovy: http://incubator.myxwiki.org/xwiki/bin/view/Sandbox/groovy
sdumitriu - (22:01): I'll add a jar now
DSpair - (22:02): sdumitriu:  And you have to add: jars="attach:myJar.jar" inside of the groovy tag.
DSpair - (22:03): Well, crazy!! It works there.
sdumitriu - (22:03): Yep
DSpair - (22:03): Is there possibly something I am missing in my Tomcat config?
sdumitriu - (22:04): http://incubator.myxwiki.org/xwiki/bin/view/Sandbox/groovy?viewer=code
sdumitriu - (22:05): So, tomcat-6.0.20, mysql, java 1.6
DSpair - (22:06): sdumitriu:  Nope, tomcat 5.5, Java 6.0.20, PostgreSQL
DSpair - (22:07): sdumitriu:  Actually, I have tried numerous combinations of Tomcat 5.5/6, Java 1.6.0.x, and PostgreSQL/MySQL on Linux and Windows.
sdumitriu - (22:08): DSpair: Voodoo
sdumitriu - (22:08): Try calling an exorcist :D
DSpair - (22:09): sdumitriu:  Hrrmmmm . . . I
DSpair - (22:09): sdumitriu:  Hrrmmmm . . . I'll get right on that . . . :P
DSpair - (22:20): sdumitriu and CalebJamesDeLisl: THANKS!!! You helped me narrow it down enough to figure it out.
CalebJamesDeLisl - (22:20): Voodoo did it? ;)
DSpair - (22:20): The problem was in the Tomcat configuration. In the server.xml file, the connector configuration has to be set to use UTF-8.
DSpair - (22:21): Once that was done, it all works perfectly.
florinciu joined #xwiki at 22:22
florinciu left at 22:23 (Client Quit
lpereira left at 22:31 (Quit: Leaving.
npm left at 22:37 (Remote host closed the connection
npm joined #xwiki at 22:37
lucaa joined #xwiki at 22:50
DSpair - (22:52): CalebJamesDeLisl:  OK, after changing the UTF-8 setting on Linux that runs fine, but on Windows it still does not work.
CalebJamesDeLisl - (22:57): Do you have an error log?
DSpair - (22:58): CalebJamesDeLisl:  Nope
DSpair - (22:58): No errors in the log.
DSpair - (22:59): I just enabled the java.security.debug and still no errors.
CalebJamesDeLisl - (23:00): I can see the possibility for a bug like this to slip by if it's only on Windxws. Didn't you have some error stack traces before?
DSpair - (23:08): CalebJamesDeLisl:  Not with attached jars.
CalebJamesDeLisl - (23:09): As far as Windxws goes I really can't help much since I dumped that software years ago.
lucaa left at 23:20 (Ping timeout: 245 seconds
sdumitriu - (23:29): DSpair: You could try to change the tomcat startup script to include en_US.UTF-8 as the system language/encoding
sdumitriu - (23:29): (on Windows)
DSpair - (23:31): sdumitriu:  Any idea on how to accomplish that?
sdumitriu - (23:34): -Dfile.encoding=UTF-8
Enygma` left at 23:45 (Ping timeout: 265 seconds
DSpair - (23:48): sdumitriu:  That did no good as either a CATALINA_OPTS or a JAVA_OPTS value.
DSpair - (23:53): I have also tried: export JAVA_OPTS="-Dfile.encoding=UTF-8 -Djavax.servlet.request.encoding=UTF-8"
DSpair - (23:54): I wonder if I need to install UTF-8 on Windows before it will work?
DSpair - (23:59): According to M$, UTF-8 is built-in as of NT4.
DSpair - (23:59): So, this appears to be a Windows specific bug.

Get Connected