From abo@dsl.dk Wed Mar 6 18:15:32 2013 Received: (at 140) by bugs.x2go.org; 6 Mar 2013 17:15:32 +0000 Received: from eu1sys200aog104.obsmtp.com (eu1sys200aog104.obsmtp.com [207.126.144.117]) by ymir (Postfix) with SMTP id 83FE95DB0D for <140@bugs.x2go.org>; Wed, 6 Mar 2013 18:15:32 +0100 (CET) Received: from mail-ia0-f200.google.com ([209.85.210.200]) (using TLSv1) by eu1sys200aob104.postini.com ([207.126.147.11]) with SMTP ID DSNKUTd5tMx4Vkff+1rUsz5/mM4v/+n0YYx+@postini.com; Wed, 06 Mar 2013 17:15:32 UTC Received: by mail-ia0-f200.google.com with SMTP id u20so31039488iag.3 for <140@bugs.x2go.org>; Wed, 06 Mar 2013 09:15:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=dsl.dk; s=google; h=x-received:mime-version:x-received:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=VjFn3DqF5YvrgOwU+mnqNE2SuCXhzr/UJHGp8pIeORc=; b=kEekJ0eNKIoZtvPyJaZiUmYDDyF8zt9xKDjVb9M/T3+W78I8hXkY+0KVJ9NAAPz3K2 e4RtSlX8BTleRE1ENg+xoU+ffVqklP9/Mp7SYLgo2LfMdd+RfzeJyHBpTPlhLpfiWveJ mUc9Ah6/0rR0qjZCnrg/+dBaNnrn1K53HAwzo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-received:mime-version:x-received:in-reply-to:references:date :message-id:subject:from:to:cc:content-type:x-gm-message-state; bh=VjFn3DqF5YvrgOwU+mnqNE2SuCXhzr/UJHGp8pIeORc=; b=InilAxmfwP0YNPuwY4skUgqQ9Od0VyE50/cXFXgcUQa9/8ZhM41ukzROR1Bfu9HlEN ONJAltaXUvvPP5jGTN5zH2A/fRy9txOEowJ7t5A9xnOVvN5zA1f8DOzlV5uRXNybeCBy CSK/Z1nKXi8Gio7N7YoHgiU4wlChfQwaWSHGtlmIw6ZK/VuEvr9vRwnNvukZtoJA+8j8 EvIp+GaLEhrNWcoYo+m6w4lK0HHWA6+79FI+PaFDR59ZaR4W6luRe0DeqWid6o/vXsns 2piVq+S31s5sfizlOqpDBWt3Te5O5AwnA7Q1gKNR5W67yCEXYgQ8XUwAZYTtGrkmjJ9W 8fiA== X-Received: by 10.182.72.5 with SMTP id z5mr23429975obu.24.1362590127407; Wed, 06 Mar 2013 09:15:27 -0800 (PST) MIME-Version: 1.0 X-Received: by 10.182.72.5 with SMTP id z5mr23429955obu.24.1362590127043; Wed, 06 Mar 2013 09:15:27 -0800 (PST) Received: by 10.182.95.211 with HTTP; Wed, 6 Mar 2013 09:15:26 -0800 (PST) In-Reply-To: <20130306180730.1027260dp4d6wdsi@mail.das-netzwerkteam.de> References: <20130306180730.1027260dp4d6wdsi@mail.das-netzwerkteam.de> Date: Wed, 6 Mar 2013 18:15:26 +0100 Message-ID: Subject: Re: [X2Go-Dev] Bug#140: x2gobroker: agent-query-mode in sessionprofiles ignored From: Anders Bruun Olsen To: Mike Gabriel Cc: 140@bugs.x2go.org Content-Type: multipart/alternative; boundary=f46d044785337f4c3204d744bc29 X-Gm-Message-State: ALoCoQls6gvIww0HvptCUyaFMHjRxNteKjCG7Y4vQfqmFNivFR3EXaMsLHNgQp4wQP0XwPGiMvXRIZYR27TXinB+D35YvnSWFGFXWlXJ2jwBrxE1bqn2wq6+BuNPuiQaF0yNuSaSURHe2YPTFcBkFV3LFWHjDM7MWg== --f46d044785337f4c3204d744bc29 Content-Type: text/plain; charset=UTF-8 2013/3/6 Mike Gabriel > > Indeed the option agent-query-mode is ignored in > x2gobroker-sessionprofiles.**conf, as it is only available in the backend > sections of x2gobroker.conf. > Right. My mistake. My understanding was "per profile", not "per backend", even though it clearly says per-broker-backend in the comment. Concept confusion I'm afraid. > I see that the new autologin (which works) uses self.get_profile_broker() >> instead of self.config to access settings. This might be the source of the >> problem? > > I could not find the usage of self.get_profile_broker() in > self.get_agent_query_mode(). If I missed something, can you point me to the > exact line? > In branch 0.0.0.6 from git: abo@x2go-broker:~/x2gobroker/x2gobroker$ grep -n self.get_profile_broker brokers/base_broker.py 478: _profile = self.get_profile_broker(profile_id) 504: _profile = self.get_profile_broker(profile_id) But that was just me trying to figure out why the autologin setting was being picked up from x2gobroker-sessionprofiles.conf while agent-query-mode wasn't. Since the answer is, that it works as intended, this last part is a non-issue as well. -- Anders Bruun Olsen It-ansvarlig Det Danske Sprog- og Litteraturselskab (Society for Danish Language and Literature) --f46d044785337f4c3204d744bc29 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable

= 2013/3/6 Mike Gabriel <mike.gabriel@das-netzwerkteam.de= >
Indeed the option agent-query-mode is ignored in x2gobroker-sessionprofiles= .conf, as it is only available in the backend sections of x2gobroker= .conf.

Right. My mistake. My unde= rstanding was "per profile", not "per backend", even th= ough it clearly says per-broker-backend in the comment. Concept confusion I= 'm afraid.
=C2=A0
I see that the new autologin (which works) uses self.get_profile_broker() instead of self.config to access settings. This might be the source of the<= br> problem?
I could not find the usage of self.get_profile_broker() in self.get_agent_q= uery_mode(). If I missed something, can you point me to the exact line?
=

In branch 0.0.0.6 from git:
abo@x2go-broker:~/x2gobroker/x2gobroker$ grep -n self.get_profile= _broker brokers/base_broker.py=C2=A0
478: =C2=A0 =C2=A0 =C2=A0 = =C2=A0_profile =3D self.get_profile_broker(profile_id)
504: =C2= =A0 =C2=A0 =C2=A0 =C2=A0_profile =3D self.get_profile_broker(profile_id)

But that was just me trying to figure out w= hy the autologin setting was being picked up from x2gobroker-sessionprofile= s.conf while agent-query-mode wasn't. Since the answer is, that it work= s as intended, this last part is a non-issue as well.

--
Anders Bruun Olsen
It-ansvarlig
Det Dansk= e Sprog- og Litteraturselskab
(Society for Danish Language and Literatur= e)
--f46d044785337f4c3204d744bc29--