Ignite Realtime: Archived bugs

Skip navigation

Up to Discussions in Tinder

4255 Views 5 Replies Latest reply: Oct 13, 2009 3:29 AM by Guus der Kinderen

andi5

10 posts since
Oct 12, 2009

Currently Being Moderated

Oct 12, 2009 10:15 AM

Archived bugs

I am interested in a fix of http://www.igniterealtime.org/issues/browse/JM-852 and suppose that

fixing it in Tinder would be correct. Does anyone still look at those bugs?

I know that it is a minor bug, but three years is quite a long time!

Like (0)

Guus der Kinderen
777 posts since
Sep 8, 2005

Currently Being Moderated

Oct 12, 2009 1:07 PM (in response to andi5)
Re: Archived bugs

Hi Andi,

The issues filed in the project you referenced are thought of as being 'archived' and therefor off our radar. This issue was no different. The issue predates the Tinder project, but is a perfect candidate to be moved into that project. This is what I just did: the issue is now known as ~~TINDER-32~~ (although the old Jira references should continue to work).

Thanks for bringing this one to our attention (again). I'll write a unit test and commit a fix hopefully later today.

Out of curiosity: did you run into this bug in practise? If so, could you share some of the details of your application of XMPP?

Regards,

Guus

Report Abuse

Like (0)
- andi5
  10 posts since
  Oct 12, 2009
  
  Currently Being Moderated
  
  Oct 13, 2009 1:39 AM (in response to Guus der Kinderen)
  Re: Archived bugs
  
  Hi Guus,
  
  that is what I call responsive, thanks a lot for fixing this issue right away!
  
  No, I did not run into this bug in practice and I suppose that will no happen soon, because our jabber ids are limited by other restrictions, like names that can be mapped to file system paths and stuff like that. I noticed it when I created a unit test that was about to create a valid huge jid in a certain situation and it simply did not work.
  
  Report Abuse
  
  Like (0)
  - Guus der Kinderen
    777 posts since
    Sep 8, 2005
    
    Currently Being Moderated
    
    Oct 13, 2009 1:45 AM (in response to andi5)
    Re: Archived bugs
    
    Ah, right. That makes sense. As you might have noticed, I've checked in a fix yesterday evening. It'd be interesting to see if this version passes your unit tests.
    
    Do you have any other unit tests that are of interest? If some of your tests cover code of Tinder that's not covered by the unit tests that are included with Tinder, I'd be interested in adding yours to the set of tests that are included with Tinder, if you allow them to be added.
    
    Report Abuse
    
    Like (0)
    - andi5
      10 posts since
      Oct 12, 2009
      
      Currently Being Moderated
      
      Oct 13, 2009 3:20 AM (in response to Guus der Kinderen)
      Re: Archived bugs
      
      Yep, the fix made my unit test work, so it is enabled by now!
      I doubt that my unit tests will be of any help to you because I am just using tinder and testing my own stuff :-)
      
      Report Abuse
      
      Like (0)
      - Guus der Kinderen
        777 posts since
        Sep 8, 2005
        
        Currently Being Moderated
        
        Oct 13, 2009 3:29 AM (in response to andi5)
        Re: Archived bugs
        
        Right. Good to hear the problem is gone now. Let me know if you run into
        something else!
        
        2009/10/13 andi5 <webmaster@igniterealtime.org>
        
        Guus der Kinderen,
        >
        A new message was posted in the thread "Archived bugs":
        >
        http://www.igniterealtime.org/community/message/196935#196935
        >
        Author : andi5
        Profile : http://www.igniterealtime.org/community/people/andi5
        >
        Message:
        
        Report Abuse
        
        Like (0)

Go to original post

Legend

Correct Answers - 10 points
Helpful Answers - 5 points

Oct 12, 2009 10:15 AM

Archived bugs

Actions

More Like This

Bookmarked By (0)

Legend