[(FBR):] pagure: Turn off debug mode on pagure.io

2019-03-07 Thread Pierre-Yves Chibon
This leads to some exception being raised instead of by-pass safely.
It was useful at some point because pagure would not log error to
its logs otherwise but Patrick has since fixed it and I believe it
may be triggering some errors now.

Signed-off-by: Pierre-Yves Chibon 
---
 roles/pagure/frontend/templates/pagure.wsgi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/roles/pagure/frontend/templates/pagure.wsgi 
b/roles/pagure/frontend/templates/pagure.wsgi
index 7f327fd0e..75c6ef343 100644
--- a/roles/pagure/frontend/templates/pagure.wsgi
+++ b/roles/pagure/frontend/templates/pagure.wsgi
@@ -27,4 +27,3 @@ os.environ['TEMP'] = '/srv/tmp/'
 from pagure.flask_app import create_app
 
 application = create_app()
-application.debug = True
-- 
2.20.1
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org


Re: [(FBR):] pagure: Turn off debug mode on pagure.io

2019-03-07 Thread Kevin Fenzi
+1

is this set in any of the other pagure instances?
(src/src.stg/pagure-stg01)?

kevin




signature.asc
Description: OpenPGP digital signature
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org


Re: [(FBR):] pagure: Turn off debug mode on pagure.io

2019-03-07 Thread Pierre-Yves Chibon
On Thu, Mar 07, 2019 at 08:11:23AM -0800, Kevin Fenzi wrote:
> +1
> 
> is this set in any of the other pagure instances?
> (src/src.stg/pagure-stg01)?

pagure.io and stg.pagure.io are covered by this change.

But indeed src and src.stg are also affected, permission to adjust the patch and
apply it there as well?


Thanks,
Pierre


signature.asc
Description: PGP signature
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org


Re: [(FBR):] pagure: Turn off debug mode on pagure.io

2019-03-07 Thread Kevin Fenzi
On 3/7/19 9:11 AM, Pierre-Yves Chibon wrote:
> On Thu, Mar 07, 2019 at 08:11:23AM -0800, Kevin Fenzi wrote:
>> +1
>>
>> is this set in any of the other pagure instances?
>> (src/src.stg/pagure-stg01)?
> 
> pagure.io and stg.pagure.io are covered by this change.
> 
> But indeed src and src.stg are also affected, permission to adjust the patch 
> and
> apply it there as well?

+1 to all of them.

kevin




signature.asc
Description: OpenPGP digital signature
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org


Re: [(FBR):] pagure: Turn off debug mode on pagure.io

2019-03-07 Thread Stephen John Smoogen
+1 also

On Thu, 7 Mar 2019 at 12:21, Kevin Fenzi  wrote:
>
> On 3/7/19 9:11 AM, Pierre-Yves Chibon wrote:
> > On Thu, Mar 07, 2019 at 08:11:23AM -0800, Kevin Fenzi wrote:
> >> +1
> >>
> >> is this set in any of the other pagure instances?
> >> (src/src.stg/pagure-stg01)?
> >
> > pagure.io and stg.pagure.io are covered by this change.
> >
> > But indeed src and src.stg are also affected, permission to adjust the 
> > patch and
> > apply it there as well?
>
> +1 to all of them.
>
> kevin
>
>
> ___
> infrastructure mailing list -- infrastructure@lists.fedoraproject.org
> To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
> Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives: 
> https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org



-- 
Stephen J Smoogen.
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org


Re: [(FBR):] pagure: Turn off debug mode on pagure.io

2019-03-08 Thread Pierre-Yves Chibon
On Thu, Mar 07, 2019 at 01:32:04PM -0500, Stephen John Smoogen wrote:
> +1 also
> 
> On Thu, 7 Mar 2019 at 12:21, Kevin Fenzi  wrote:
> >
> > On 3/7/19 9:11 AM, Pierre-Yves Chibon wrote:
> > > On Thu, Mar 07, 2019 at 08:11:23AM -0800, Kevin Fenzi wrote:
> > >> +1
> > >>
> > >> is this set in any of the other pagure instances?
> > >> (src/src.stg/pagure-stg01)?
> > >
> > > pagure.io and stg.pagure.io are covered by this change.
> > >
> > > But indeed src and src.stg are also affected, permission to adjust the 
> > > patch and
> > > apply it there as well?
> >
> > +1 to all of them.

Applied, thanks :)


Pierre
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org