<feed xmlns='http://www.w3.org/2005/Atom'>
<title>ipsilon.git/ipsilon, branch log_location</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/'/>
<entry>
<title>Add source code context information to debug log messages</title>
<updated>2015-01-21T19:04:12+00:00</updated>
<author>
<name>John Dennis</name>
<email>jdennis@redhat.com</email>
</author>
<published>2015-01-12T15:47:37+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=00d5e9272667605c6b5bb83746dcdea54bfa3977'/>
<id>00d5e9272667605c6b5bb83746dcdea54bfa3977</id>
<content type='text'>
The log.debug() function helpfully adds the name of the function
invoking it but in a complicated software package there are many
functions/methods which share the same name. Thus a debug message lke
this:

DEBUG(__init__): xxx

does not give you much context, there are probably hundreds of
__init__ methods. It would help to qualify the method name which it's
class name, that gives a lot more context when reading the
log. Sometimes it's also helpful to know the file and line number.

This patch adds the class name to the function and included the
filename and line number as well. The file path is trimmed to the last
3 components, sufficient to give context but not too verbose. Now the
debug message might look like this instead:

DEBUG(ipsilon/providers/common.py:129 LoadProviders.__init__()): xxx

Also included is a config option 'stacktrace_on_error' which will
include a stacktrace when the log.error function is called. It can be
very useful to see a stacktrace when logging an error, it defaults to
off.

Signed-off-by: John Dennis &lt;jdennis@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The log.debug() function helpfully adds the name of the function
invoking it but in a complicated software package there are many
functions/methods which share the same name. Thus a debug message lke
this:

DEBUG(__init__): xxx

does not give you much context, there are probably hundreds of
__init__ methods. It would help to qualify the method name which it's
class name, that gives a lot more context when reading the
log. Sometimes it's also helpful to know the file and line number.

This patch adds the class name to the function and included the
filename and line number as well. The file path is trimmed to the last
3 components, sufficient to give context but not too verbose. Now the
debug message might look like this instead:

DEBUG(ipsilon/providers/common.py:129 LoadProviders.__init__()): xxx

Also included is a config option 'stacktrace_on_error' which will
include a stacktrace when the log.error function is called. It can be
very useful to see a stacktrace when logging an error, it defaults to
off.

Signed-off-by: John Dennis &lt;jdennis@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix file permissions and remove shebang's</title>
<updated>2014-12-16T15:51:43+00:00</updated>
<author>
<name>Patrick Uiterwijk</name>
<email>puiterwijk@redhat.com</email>
</author>
<published>2014-12-16T15:40:03+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=45cb73a21a90084818c3057e362ef9459f1600f3'/>
<id>45cb73a21a90084818c3057e362ef9459f1600f3</id>
<content type='text'>
Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Make pep8 happy again</title>
<updated>2014-12-05T20:55:35+00:00</updated>
<author>
<name>Simo Sorce</name>
<email>simo@redhat.com</email>
</author>
<published>2014-12-05T20:54:02+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=49553b2d416566fabed0d80b31e7218c8abde7a0'/>
<id>49553b2d416566fabed0d80b31e7218c8abde7a0</id>
<content type='text'>
Signed-off-by: Simo Sorce &lt;simo@redhat.com&gt;
Reviewed-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Simo Sorce &lt;simo@redhat.com&gt;
Reviewed-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Add OpenIDStore to store associations and nonces</title>
<updated>2014-12-05T20:44:17+00:00</updated>
<author>
<name>Patrick Uiterwijk</name>
<email>puiterwijk@redhat.com</email>
</author>
<published>2014-12-05T17:28:21+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=a7232b6ff6edfcbe36dd44fc0419a71099fda4cb'/>
<id>a7232b6ff6edfcbe36dd44fc0419a71099fda4cb</id>
<content type='text'>
Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Add defaults to List objects</title>
<updated>2014-12-05T20:44:06+00:00</updated>
<author>
<name>Simo Sorce</name>
<email>simo@redhat.com</email>
</author>
<published>2014-12-05T20:37:28+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=6b37bf2965a5c66cbe0f2a1def92a815170c4ce3'/>
<id>6b37bf2965a5c66cbe0f2a1def92a815170c4ce3</id>
<content type='text'>
Otherwise we get backtraces when checking for list members and no configuration
have been stored in the database yet.

Signed-off-by: Simo Sorce &lt;simo@redhat.com&gt;
Reviewed-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Otherwise we get backtraces when checking for list members and no configuration
have been stored in the database yet.

Signed-off-by: Simo Sorce &lt;simo@redhat.com&gt;
Reviewed-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Allow to pass drectly a URL to the Store class</title>
<updated>2014-12-05T19:49:30+00:00</updated>
<author>
<name>Simo Sorce</name>
<email>simo@redhat.com</email>
</author>
<published>2014-12-05T19:28:22+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=1c3343573f008e7fe4c14cfb3523960a5376dc30'/>
<id>1c3343573f008e7fe4c14cfb3523960a5376dc30</id>
<content type='text'>
This is useful for plugins that want to use their own database configuration
but still want to reuse he Store class for simplicity.

Signed-off-by: Simo Sorce &lt;simo@redhat.com&gt;
Reviewed-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is useful for plugins that want to use their own database configuration
but still want to reuse he Store class for simplicity.

Signed-off-by: Simo Sorce &lt;simo@redhat.com&gt;
Reviewed-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Add support for Persona Identity Provider</title>
<updated>2014-11-14T18:06:27+00:00</updated>
<author>
<name>Patrick Uiterwijk</name>
<email>puiterwijk@redhat.com</email>
</author>
<published>2014-11-13T09:18:05+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=943158d19f879eb6ad515edeb59017671e4252c5'/>
<id>943158d19f879eb6ad515edeb59017671e4252c5</id>
<content type='text'>
Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Make sure the XRDS is returned as string</title>
<updated>2014-11-13T14:41:32+00:00</updated>
<author>
<name>Patrick Uiterwijk</name>
<email>puiterwijk@redhat.com</email>
</author>
<published>2014-11-13T13:45:13+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=d4f4bc7b1ed80887534698825fc93ff3cf68dfe7'/>
<id>d4f4bc7b1ed80887534698825fc93ff3cf68dfe7</id>
<content type='text'>
Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Delay exposing OpenID</title>
<updated>2014-11-13T14:40:27+00:00</updated>
<author>
<name>Patrick Uiterwijk</name>
<email>puiterwijk@redhat.com</email>
</author>
<published>2014-11-13T13:39:30+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=84c689d0420a4e16ccdd30f55c8cec0882c0495d'/>
<id>84c689d0420a4e16ccdd30f55c8cec0882c0495d</id>
<content type='text'>
This makes sure we have loaded the configuration
before using it

Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This makes sure we have loaded the configuration
before using it

Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix LDAP plugin configuration checks</title>
<updated>2014-11-13T14:32:01+00:00</updated>
<author>
<name>Patrick Uiterwijk</name>
<email>puiterwijk@redhat.com</email>
</author>
<published>2014-11-13T12:59:41+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/jdennis/public_git/ipsilon.git/commit/?id=05ab5acd2a57c273d06c5aa1176d2e9da6e7c824'/>
<id>05ab5acd2a57c273d06c5aa1176d2e9da6e7c824</id>
<content type='text'>
Interpret config value correctly (it is a boolean now)
Pass required argument

Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Interpret config value correctly (it is a boolean now)
Pass required argument

Signed-off-by: Patrick Uiterwijk &lt;puiterwijk@redhat.com&gt;
Reviewed-by: Simo Sorce &lt;simo@redhat.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
