Re: [rhino-tools-dev] Re: Rhino-Security and second-level cache
Any news for this issues? I am running also into this problem. On Friday, July 29, 2011 8:50:06 PM UTC+3, Nathan wrote: > > No, it is not > > On Fri, Jul 29, 2011 at 12:18 PM, Bhoomi > > wrote: > >> Is this issue resolved? >> >> I am facing the same issue. >> >> >> On Fri, May 6, 2011 at 9:25 PM, Nathan Stott >> > wrote: >> >>> I made a comment about your pull request and you replied to it, but >>> never changed what we talked about in the comments. >>> >>> On Fri, May 6, 2011 at 3:45 AM, vandalo > >>> wrote: >>> > I don't understand who's managing this project. I've pulled a request >>> > and submitted some changes (with test-case as required) but it's just >>> > on my account. >>> > Nobody replied or bothered to apply these changes. >>> > Now I was looking at RemCoros repository and found out that he made >>> > some changes but they do exist only in it's project. >>> > >>> > Alberto >>> > >>> > On Apr 22, 11:33 am, Nathan Stott wrote: >>> >> Can you send a pull request? The only recent pull request I've >>> >> received is this one: >>> https://github.com/vandalo/rhino-security/commit/0795e30ee8b7dd8651ad... >>> >> >>> >> >>> >> >>> >> >>> >> >>> >> >>> >> >>> >> On Fri, Apr 22, 2011 at 2:31 AM, Remco Ros wrote: >>> >> > Hi Rick, >>> >> >>> >> > I have never used rhino-security, so I can't be 100% sure about the >>> >> > rammification. But as far as I can see the only place where data >>> access code >>> >> > lives is in that repository. >>> >> >>> >> > There are two options: >>> >> > - evict the whole query cache after each change (this has some >>> performance >>> >> > implications when you're doing a lot of updates/reads at the same >>> time, >>> >> > because after each update the whole query cache is evicted) >>> >> > - split the queries in different cache regions (using >>> SetCacheRegion) and >>> >> > evict that regions query cache whenever an update occurs which >>> should >>> >> > invalidate that region. >>> >> >>> >> > If you can write the failing tests, I can implement selective query >>> eviction >>> >> > like I did for the first test you wrote. >>> >> > (https://github.com/remcoros/rhino-security) >>> >> >>> >> > Also maybe Nathan can have a look at this? >>> >> >>> >> > Remco >>> >> >>> >> > -- >>> >> > You received this message because you are subscribed to the Google >>> Groups >>> >> > "Rhino Tools Dev" group. >>> >> > To post to this group, send email to >>> >> > [email protected] >>> . >>> >> > To unsubscribe from this group, send email to >>> >> > [email protected] . >>> >> > For more options, visit this group at >>> >> >http://groups.google.com/group/rhino-tools-dev?hl=en. >>> > >>> > -- >>> > You received this message because you are subscribed to the Google >>> Groups "Rhino Tools Dev" group. >>> > To post to this group, send email to >>> > [email protected] >>> . >>> > To unsubscribe from this group, send email to >>> [email protected] . >>> > For more options, visit this group at >>> http://groups.google.com/group/rhino-tools-dev?hl=en. >>> > >>> > >>> >>> -- >>> You received this message because you are subscribed to the Google >>> Groups "Rhino Tools Dev" group. >>> To post to this group, send email to >>> [email protected] >>> . >>> To unsubscribe from this group, send email to >>> [email protected] . >>> For more options, visit this group at >>> http://groups.google.com/group/rhino-tools-dev?hl=en. >>> >>> >> -- >> You received this message because you are subscribed to the Google Groups >> "Rhino Tools Dev" group. >> To post to this group, send email to [email protected] >> . >> To unsubscribe from this group, send email to >> [email protected] . >> For more options, visit this group at >> http://groups.google.com/group/rhino-tools-dev?hl=en. >> > > -- You received this message because you are subscribed to the Google Groups "Rhino Tools Dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/rhino-tools-dev?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
Re: [rhino-tools-dev] Re: Rhino-Security and second-level cache
No, it is not On Fri, Jul 29, 2011 at 12:18 PM, Bhoomi wrote: > Is this issue resolved? > > I am facing the same issue. > > > On Fri, May 6, 2011 at 9:25 PM, Nathan Stott wrote: > >> I made a comment about your pull request and you replied to it, but >> never changed what we talked about in the comments. >> >> On Fri, May 6, 2011 at 3:45 AM, vandalo wrote: >> > I don't understand who's managing this project. I've pulled a request >> > and submitted some changes (with test-case as required) but it's just >> > on my account. >> > Nobody replied or bothered to apply these changes. >> > Now I was looking at RemCoros repository and found out that he made >> > some changes but they do exist only in it's project. >> > >> > Alberto >> > >> > On Apr 22, 11:33 am, Nathan Stott wrote: >> >> Can you send a pull request? The only recent pull request I've >> >> received is this one: >> https://github.com/vandalo/rhino-security/commit/0795e30ee8b7dd8651ad... >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> >> On Fri, Apr 22, 2011 at 2:31 AM, Remco Ros wrote: >> >> > Hi Rick, >> >> >> >> > I have never used rhino-security, so I can't be 100% sure about the >> >> > rammification. But as far as I can see the only place where data >> access code >> >> > lives is in that repository. >> >> >> >> > There are two options: >> >> > - evict the whole query cache after each change (this has some >> performance >> >> > implications when you're doing a lot of updates/reads at the same >> time, >> >> > because after each update the whole query cache is evicted) >> >> > - split the queries in different cache regions (using SetCacheRegion) >> and >> >> > evict that regions query cache whenever an update occurs which should >> >> > invalidate that region. >> >> >> >> > If you can write the failing tests, I can implement selective query >> eviction >> >> > like I did for the first test you wrote. >> >> > (https://github.com/remcoros/rhino-security) >> >> >> >> > Also maybe Nathan can have a look at this? >> >> >> >> > Remco >> >> >> >> > -- >> >> > You received this message because you are subscribed to the Google >> Groups >> >> > "Rhino Tools Dev" group. >> >> > To post to this group, send email to >> [email protected]. >> >> > To unsubscribe from this group, send email to >> >> > [email protected]. >> >> > For more options, visit this group at >> >> >http://groups.google.com/group/rhino-tools-dev?hl=en. >> > >> > -- >> > You received this message because you are subscribed to the Google >> Groups "Rhino Tools Dev" group. >> > To post to this group, send email to [email protected]. >> > To unsubscribe from this group, send email to >> [email protected]. >> > For more options, visit this group at >> http://groups.google.com/group/rhino-tools-dev?hl=en. >> > >> > >> >> -- >> You received this message because you are subscribed to the Google Groups >> "Rhino Tools Dev" group. >> To post to this group, send email to [email protected]. >> To unsubscribe from this group, send email to >> [email protected]. >> For more options, visit this group at >> http://groups.google.com/group/rhino-tools-dev?hl=en. >> >> > -- > You received this message because you are subscribed to the Google Groups > "Rhino Tools Dev" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]. > For more options, visit this group at > http://groups.google.com/group/rhino-tools-dev?hl=en. > -- You received this message because you are subscribed to the Google Groups "Rhino Tools Dev" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rhino-tools-dev?hl=en.
Re: [rhino-tools-dev] Re: Rhino-Security and second-level cache
Is this issue resolved? I am facing the same issue. On Fri, May 6, 2011 at 9:25 PM, Nathan Stott wrote: > I made a comment about your pull request and you replied to it, but > never changed what we talked about in the comments. > > On Fri, May 6, 2011 at 3:45 AM, vandalo wrote: > > I don't understand who's managing this project. I've pulled a request > > and submitted some changes (with test-case as required) but it's just > > on my account. > > Nobody replied or bothered to apply these changes. > > Now I was looking at RemCoros repository and found out that he made > > some changes but they do exist only in it's project. > > > > Alberto > > > > On Apr 22, 11:33 am, Nathan Stott wrote: > >> Can you send a pull request? The only recent pull request I've > >> received is this one: > https://github.com/vandalo/rhino-security/commit/0795e30ee8b7dd8651ad... > >> > >> > >> > >> > >> > >> > >> > >> On Fri, Apr 22, 2011 at 2:31 AM, Remco Ros wrote: > >> > Hi Rick, > >> > >> > I have never used rhino-security, so I can't be 100% sure about the > >> > rammification. But as far as I can see the only place where data > access code > >> > lives is in that repository. > >> > >> > There are two options: > >> > - evict the whole query cache after each change (this has some > performance > >> > implications when you're doing a lot of updates/reads at the same > time, > >> > because after each update the whole query cache is evicted) > >> > - split the queries in different cache regions (using SetCacheRegion) > and > >> > evict that regions query cache whenever an update occurs which should > >> > invalidate that region. > >> > >> > If you can write the failing tests, I can implement selective query > eviction > >> > like I did for the first test you wrote. > >> > (https://github.com/remcoros/rhino-security) > >> > >> > Also maybe Nathan can have a look at this? > >> > >> > Remco > >> > >> > -- > >> > You received this message because you are subscribed to the Google > Groups > >> > "Rhino Tools Dev" group. > >> > To post to this group, send email to [email protected] > . > >> > To unsubscribe from this group, send email to > >> > [email protected]. > >> > For more options, visit this group at > >> >http://groups.google.com/group/rhino-tools-dev?hl=en. > > > > -- > > You received this message because you are subscribed to the Google Groups > "Rhino Tools Dev" group. > > To post to this group, send email to [email protected]. > > To unsubscribe from this group, send email to > [email protected]. > > For more options, visit this group at > http://groups.google.com/group/rhino-tools-dev?hl=en. > > > > > > -- > You received this message because you are subscribed to the Google Groups > "Rhino Tools Dev" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]. > For more options, visit this group at > http://groups.google.com/group/rhino-tools-dev?hl=en. > > -- You received this message because you are subscribed to the Google Groups "Rhino Tools Dev" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rhino-tools-dev?hl=en.
Re: [rhino-tools-dev] Re: Rhino-Security and second-level cache
I made a comment about your pull request and you replied to it, but never changed what we talked about in the comments. On Fri, May 6, 2011 at 3:45 AM, vandalo wrote: > I don't understand who's managing this project. I've pulled a request > and submitted some changes (with test-case as required) but it's just > on my account. > Nobody replied or bothered to apply these changes. > Now I was looking at RemCoros repository and found out that he made > some changes but they do exist only in it's project. > > Alberto > > On Apr 22, 11:33 am, Nathan Stott wrote: >> Can you send a pull request? The only recent pull request I've >> received is this >> one:https://github.com/vandalo/rhino-security/commit/0795e30ee8b7dd8651ad... >> >> >> >> >> >> >> >> On Fri, Apr 22, 2011 at 2:31 AM, Remco Ros wrote: >> > Hi Rick, >> >> > I have never used rhino-security, so I can't be 100% sure about the >> > rammification. But as far as I can see the only place where data access >> > code >> > lives is in that repository. >> >> > There are two options: >> > - evict the whole query cache after each change (this has some performance >> > implications when you're doing a lot of updates/reads at the same time, >> > because after each update the whole query cache is evicted) >> > - split the queries in different cache regions (using SetCacheRegion) and >> > evict that regions query cache whenever an update occurs which should >> > invalidate that region. >> >> > If you can write the failing tests, I can implement selective query >> > eviction >> > like I did for the first test you wrote. >> > (https://github.com/remcoros/rhino-security) >> >> > Also maybe Nathan can have a look at this? >> >> > Remco >> >> > -- >> > You received this message because you are subscribed to the Google Groups >> > "Rhino Tools Dev" group. >> > To post to this group, send email to [email protected]. >> > To unsubscribe from this group, send email to >> > [email protected]. >> > For more options, visit this group at >> >http://groups.google.com/group/rhino-tools-dev?hl=en. > > -- > You received this message because you are subscribed to the Google Groups > "Rhino Tools Dev" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]. > For more options, visit this group at > http://groups.google.com/group/rhino-tools-dev?hl=en. > > -- You received this message because you are subscribed to the Google Groups "Rhino Tools Dev" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rhino-tools-dev?hl=en.
Re: [rhino-tools-dev] Re: Rhino-Security and second-level cache
Can you send a pull request? The only recent pull request I've received is this one: https://github.com/vandalo/rhino-security/commit/0795e30ee8b7dd8651adc443190a98aee6befc1e#comments On Fri, Apr 22, 2011 at 2:31 AM, Remco Ros wrote: > Hi Rick, > > I have never used rhino-security, so I can't be 100% sure about the > rammification. But as far as I can see the only place where data access code > lives is in that repository. > > There are two options: > - evict the whole query cache after each change (this has some performance > implications when you're doing a lot of updates/reads at the same time, > because after each update the whole query cache is evicted) > - split the queries in different cache regions (using SetCacheRegion) and > evict that regions query cache whenever an update occurs which should > invalidate that region. > > If you can write the failing tests, I can implement selective query eviction > like I did for the first test you wrote. > (https://github.com/remcoros/rhino-security) > > Also maybe Nathan can have a look at this? > > Remco > > -- > You received this message because you are subscribed to the Google Groups > "Rhino Tools Dev" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]. > For more options, visit this group at > http://groups.google.com/group/rhino-tools-dev?hl=en. > -- You received this message because you are subscribed to the Google Groups "Rhino Tools Dev" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rhino-tools-dev?hl=en.
Re: [rhino-tools-dev] Re: Rhino-Security and second-level cache
I'm home. My use of rhino security is in projects that are in the
maintenance phase, not the new development phase. I will be happy to
apply any patches you create; however, if you're waiting on me to dig
into an issue that doesn't directly involve maintaining one of my
existing sites using rhino security, you're going to be waiting for
quite a while.
Good luck and if you dig into the issue and resolve it, I'll be happy
to apply your commit.
On Fri, Mar 25, 2011 at 5:37 PM, RickB wrote:
> Hello? Is anyone home?
>
> On Mar 11, 5:23 pm, RickB wrote:
>> It looks like group.Users.Remove(user) (Services
>> \AuthorizationRepository.cs line 525) is not actually removing the
>> user from the group. I don't know enough about how the system works
>> under the hood to come up with a fix.
>>
>> Do you think you could please dig into it a bit? I bet it's an easy
>> fix for someone who is familiar with how it works.
>>
>> Thank you!
>>
>> Rick
>>
>> On Mar 10, 8:09 am, Nathan Stott wrote:
>>
>>
>>
>> > It has yet to be fixed. If you'd like to submit a patch and a pull
>> > request I will happily merge it.
>>
>> > On Mon, Mar 7, 2011 at 5:44 PM, RickB wrote:
>> > > Does anyone know if this issue has been fixed?
>>
>> > > Thank you,
>>
>> > > Rick
>>
>> > > On Feb 25, 12:14 am, vandalo wrote:
>> > >> Thanks David,
>>
>> > >> Alberto
>>
>> > >> On Feb 23, 11:58 pm, David Archer wrote:
>>
>> > >> > It's a known issue -- Google it and you'll see others are having the
>> > >> > same problem. I'm not experienced enough with NH to know what the root
>> > >> > cause is here. :(
>>
>> > >> > I have however just submitted a pull request on Github to Ayende with
>> > >> > a failing unit test illustrating the problem.
>>
>> > >> >https://github.com/ayende/rhino-security/pull/8
>>
>> > >> > -- David Archer
>>
>> > >> > On Feb 15, 11:14 am, vandalo wrote:
>>
>> > >> > > I am pretty sure that this question has been asked many times but I
>> > >> > > haven't been able to find an answer yet.
>> > >> > > DetachUserFromGroup and/or AssociateUserWith members of
>> > >> > > AuthorizationRepository don't seem to invalidate the second-level
>> > >> > > cache.
>> > >> > > I've tried to investigate more but my knowledge of Nhibernate is too
>> > >> > > poor at the moment to find a proper solution.
>> > >> > > I've downloaded Rhino-Security from
>> > >> > > here:https://github.com/ayende/rhino-security
>> > >> > > and extended AuthorizationServiceWithSecondLevelCacheFixture.
>>
>> > >> > > The DatabaseFixture basically creates a user, 3 groups and associate
>> > >> > > the user with all the groups:
>>
>> > >> > > user = new User { Name = "Ayende" };
>>
>> > >> > > session.Save(user);
>>
>> > >> > > authorizationService =
>> > >> > > ServiceLocator.Current.GetInstance();
>> > >> > > permissionService =
>> > >> > > ServiceLocator.Current.GetInstance();
>> > >> > > permissionsBuilderService =
>> > >> > > ServiceLocator.Current.GetInstance();
>> > >> > > authorizationRepository =
>> > >> > > ServiceLocator.Current.GetInstance();
>>
>> > >> > > authorizationRepository.CreateUsersGroup("Administrators");
>> > >> > > authorizationRepository.CreateUsersGroup("Users");
>> > >> > > authorizationRepository.CreateUsersGroup("Guests");
>>
>> > >> > > authorizationRepository.AssociateUserWith(user, "Administrators");
>> > >> > > authorizationRepository.AssociateUserWith(user, "Users");
>> > >> > > authorizationRepository.AssociateUserWith(user, "Guests");
>>
>> > >> > > I've added this fixture
>>
>> > >> > > [Fact]
>> > >> > > public void Test_Associated_Users_Group_For()
>> > >> > > {
>>
>> > >> > > session.Flush();
>> > >> > > session.Transaction.Commit();
>> > >> > > session.Dispose();
>>
>> > >> > > using (var s2 = factory.OpenSession())
>> > >> > > {
>> > >> > > using (var tx = s2.BeginTransaction())
>> > >> > > {
>> > >> > > SillyContainer.SessionProvider = () => s2;
>>
>> > >> > > var User = s2.CreateCriteria()
>> > >> > > .Add(Restrictions.Eq("Name", "Ayende"))
>> > >> > > .UniqueResult();
>>
>> > >> > > var anotherAuthorizationRepository =
>> > >> > > ServiceLocator.Current.GetInstance();
>>
>> > >> > > var Groups =
>> > >> > > anotherAuthorizationRepository.GetAssociatedUsersGroupFor(User);
>>
>> > >> > > anotherAuthorizationRepository.DetachUserFromGroup(User,
>> > >> > > "Guests");
>>
>> > >> > > s2.Flush();
>> > >> > > tx.Commit();
>> > >> > > }
>> > >> > > }
>> > >> > > using (var s3 = factory.OpenSession())
>> > >> > > {
>> > >> > > using (var tx = s3.BeginTransaction())
>> > >> > > {
>> > >> > > SillyContainer.SessionProvider = () => s3;
>>
>> > >> > > var User = s3.CreateCriteria()
>> > >> > > .Add(Restrictions.Eq("Name", "Ayende"))
>> > >> > > .UniqueResult();
>>
>> > >> > > var
Re: [rhino-tools-dev] Re: Rhino-Security and second-level cache
It has yet to be fixed. If you'd like to submit a patch and a pull
request I will happily merge it.
On Mon, Mar 7, 2011 at 5:44 PM, RickB wrote:
> Does anyone know if this issue has been fixed?
>
> Thank you,
>
> Rick
>
> On Feb 25, 12:14 am, vandalo wrote:
>> Thanks David,
>>
>> Alberto
>>
>> On Feb 23, 11:58 pm, David Archer wrote:
>>
>>
>>
>> > It's a known issue -- Google it and you'll see others are having the
>> > same problem. I'm not experienced enough with NH to know what the root
>> > cause is here. :(
>>
>> > I have however just submitted a pull request on Github to Ayende with
>> > a failing unit test illustrating the problem.
>>
>> >https://github.com/ayende/rhino-security/pull/8
>>
>> > -- David Archer
>>
>> > On Feb 15, 11:14 am, vandalo wrote:
>>
>> > > I am pretty sure that this question has been asked many times but I
>> > > haven't been able to find an answer yet.
>> > > DetachUserFromGroup and/or AssociateUserWith members of
>> > > AuthorizationRepository don't seem to invalidate the second-level
>> > > cache.
>> > > I've tried to investigate more but my knowledge of Nhibernate is too
>> > > poor at the moment to find a proper solution.
>> > > I've downloaded Rhino-Security from
>> > > here:https://github.com/ayende/rhino-security
>> > > and extended AuthorizationServiceWithSecondLevelCacheFixture.
>>
>> > > The DatabaseFixture basically creates a user, 3 groups and associate
>> > > the user with all the groups:
>>
>> > > user = new User { Name = "Ayende" };
>>
>> > > session.Save(user);
>>
>> > > authorizationService =
>> > > ServiceLocator.Current.GetInstance();
>> > > permissionService =
>> > > ServiceLocator.Current.GetInstance();
>> > > permissionsBuilderService =
>> > > ServiceLocator.Current.GetInstance();
>> > > authorizationRepository =
>> > > ServiceLocator.Current.GetInstance();
>>
>> > > authorizationRepository.CreateUsersGroup("Administrators");
>> > > authorizationRepository.CreateUsersGroup("Users");
>> > > authorizationRepository.CreateUsersGroup("Guests");
>>
>> > > authorizationRepository.AssociateUserWith(user, "Administrators");
>> > > authorizationRepository.AssociateUserWith(user, "Users");
>> > > authorizationRepository.AssociateUserWith(user, "Guests");
>>
>> > > I've added this fixture
>>
>> > > [Fact]
>> > > public void Test_Associated_Users_Group_For()
>> > > {
>>
>> > > session.Flush();
>> > > session.Transaction.Commit();
>> > > session.Dispose();
>>
>> > > using (var s2 = factory.OpenSession())
>> > > {
>> > > using (var tx = s2.BeginTransaction())
>> > > {
>> > > SillyContainer.SessionProvider = () => s2;
>>
>> > > var User = s2.CreateCriteria()
>> > > .Add(Restrictions.Eq("Name", "Ayende"))
>> > > .UniqueResult();
>>
>> > > var anotherAuthorizationRepository =
>> > > ServiceLocator.Current.GetInstance();
>>
>> > > var Groups =
>> > > anotherAuthorizationRepository.GetAssociatedUsersGroupFor(User);
>>
>> > > anotherAuthorizationRepository.DetachUserFromGroup(User,
>> > > "Guests");
>>
>> > > s2.Flush();
>> > > tx.Commit();
>> > > }
>> > > }
>> > > using (var s3 = factory.OpenSession())
>> > > {
>> > > using (var tx = s3.BeginTransaction())
>> > > {
>> > > SillyContainer.SessionProvider = () => s3;
>>
>> > > var User = s3.CreateCriteria()
>> > > .Add(Restrictions.Eq("Name", "Ayende"))
>> > > .UniqueResult();
>>
>> > > var anotherAuthorizationRepository =
>> > > ServiceLocator.Current.GetInstance();
>> > > var Groups =
>> > > anotherAuthorizationRepository.GetAssociatedUsersGroupFor(User);
>>
>> > > tx.Commit();
>> > > }
>> > > }
>>
>> > > }
>>
>> > > The 2nd session detaches the user from the group Guests but the 3rd
>> > > session still fetches 3 groups associated with the user.
>> > > It seems that the second-level cache can't be invalidated.
>> > > Is there anyone who can help me?- Hide quoted text -
>>
>> - Show quoted text -
>
> --
> You received this message because you are subscribed to the Google Groups
> "Rhino Tools Dev" group.
> To post to this group, send email to [email protected].
> To unsubscribe from this group, send email to
> [email protected].
> For more options, visit this group at
> http://groups.google.com/group/rhino-tools-dev?hl=en.
>
>
--
You received this message because you are subscribed to the Google Groups
"Rhino Tools Dev" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/rhino-tools-dev?hl=en.
