[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #22 from Tomás Cohen Arazi  ---
Created attachment 87545
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=87545=edit
[UGLY FIX] Bug 18205: Set userenv in
Koha::REST::V1::Auth::authenticate_api_request

Koha code (like C4::Log::logaction) relies on C4::Context::userenv to be
set. This could be fixed in many ways. The easiest way is to set that
environmental variables after authorization happens.

I don't think this is a good design, but I'm not sure adding a new param
to logaction is the best option.

I'd rather make the action_logs table another target for Log4Perl at
some point. But I strongly disagree on making tiny enhancements be
required to refactor all the things :-D

I drop this code here for others to weight in.

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Tomás Cohen Arazi  changed:

   What|Removed |Added

 QA Contact||tomasco...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Tomás Cohen Arazi  changed:

   What|Removed |Added

 CC||kyle.m.h...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Tomás Cohen Arazi  changed:

   What|Removed |Added

 CC||martin.renvoize@ptfs-europe
   ||.com
 Status|Signed Off  |Failed QA

--- Comment #21 from Tomás Cohen Arazi  ---
I've:
- Renamed 'rest' for 'api'
- Added some more tests for V1.pm.

I found that we actually don't set userenv. The test highlights this. I will
provide a patch to set it (as we do with 'interface') but I want to hear
other's opinions.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #20 from Tomás Cohen Arazi  ---
Created attachment 87542
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=87542=edit
Bug 18205: (follow-up) Tests for V1 setting the right interface

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #19 from Tomás Cohen Arazi  ---
Created attachment 87541
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=87541=edit
Bug 18205: (QA follow-up) Use 'api'

Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #17 from Tomás Cohen Arazi  ---
Created attachment 87539
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=87539=edit
Bug 18205: Mojo::Log with Koha::Logger::Mojo

This patch adds a new module, Koha::Logger::Mojo, based on
MojoX::Log::Log4perl.
This module allows us to use log4perl configurations with Mojolicious on the
freshly introduced 'rest' interface in log4perl.conf.

$c->app->log->warn will then use Log4perl as the underlying log mechanism and
those log events will point to the appropriate log file.

To test:
1. Install Perl dependency MojoX::Log::Log4perl
2. prove t/Koha/Logger/Mojo.t

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #18 from Tomás Cohen Arazi  ---
Created attachment 87540
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=87540=edit
Bug 18205: (follow-up) Fix pod in Koha::Logger::Mojo

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #15 from Tomás Cohen Arazi  ---
Created attachment 87537
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=87537=edit
Bug 18205: Add interface 'rest' for Koha::Logger

This patch adds an interface 'rest' for Koha::Logger.

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #16 from Tomás Cohen Arazi  ---
Created attachment 87538
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=87538=edit
Bug 18205: Unit tests

To test:
1. prove t/db_dependent/Log.t

Signed-off-by: Josef Moravec 
Signed-off-by: Tomas Cohen Arazi 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #85713|0   |1
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Tomás Cohen Arazi  changed:

   What|Removed |Added

 CC||tomasco...@gmail.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #85711|0   |1
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #85714|0   |1
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Tomás Cohen Arazi  changed:

   What|Removed |Added

  Attachment #85712|0   |1
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Katrin Fischer  changed:

   What|Removed |Added

 QA Contact|testo...@bugs.koha-communit |
   |y.org   |
  Component|Web services|REST api
 CC||katrin.fisc...@bsz-bw.de

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #13 from Josef Moravec  ---
Created attachment 85713
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85713=edit
Bug 18205: Mojo::Log with Koha::Logger::Mojo

This patch adds a new module, Koha::Logger::Mojo, based on
MojoX::Log::Log4perl.
This module allows us to use log4perl configurations with Mojolicious on the
freshly introduced 'rest' interface in log4perl.conf.

$c->app->log->warn will then use Log4perl as the underlying log mechanism and
those log events will point to the appropriate log file.

To test:
1. Install Perl dependency MojoX::Log::Log4perl
2. prove t/Koha/Logger/Mojo.t

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #12 from Josef Moravec  ---
Created attachment 85712
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85712=edit
Bug 18205: Unit tests

To test:
1. prove t/db_dependent/Log.t

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Josef Moravec  changed:

   What|Removed |Added

  Attachment #69226|0   |1
is obsolete||
  Attachment #69227|0   |1
is obsolete||
  Attachment #69232|0   |1
is obsolete||

--- Comment #11 from Josef Moravec  ---
Created attachment 85711
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85711=edit
Bug 18205: Add interface 'rest' for Koha::Logger

This patch adds an interface 'rest' for Koha::Logger.

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #14 from Josef Moravec  ---
Created attachment 85714
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=85714=edit
Bug 18205: (follow-up) Fix pod in Koha::Logger::Mojo

Signed-off-by: Josef Moravec 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Josef Moravec  changed:

   What|Removed |Added

 Status|Patch doesn't apply |Signed Off

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

M. Tompsett  changed:

   What|Removed |Added

 CC||mtomp...@hotmail.com

--- Comment #10 from M. Tompsett  ---
This still hasn't been rebased, so not sure if I can test bug 18206.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Michal Denar  changed:

   What|Removed |Added

 CC||blac...@gmail.com
 Status|Needs Signoff   |Patch doesn't apply

--- Comment #9 from Michal Denar  ---
Still needed? If yes, please rebase on master.

Thanks

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Lari Taskula  changed:

   What|Removed |Added

  Attachment #69228|0   |1
is obsolete||

--- Comment #8 from Lari Taskula  ---
Created attachment 69232
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69232=edit
Bug 18205: Mojo::Log with Koha::Logger::Mojo

This patch adds a new module, Koha::Logger::Mojo, based on
MojoX::Log::Log4perl.
This module allows us to use log4perl configurations with Mojolicious on the
freshly introduced 'rest' interface in log4perl.conf.

$c->app->log->warn will then use Log4perl as the underlying log mechanism and
those log events will point to the appropriate log file.

To test:
1. Install Perl dependency MojoX::Log::Log4perl
2. prove t/Koha/Logger/Mojo.t

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Lari Taskula  changed:

   What|Removed |Added

 Status|Patch doesn't apply |Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Lari Taskula  changed:

   What|Removed |Added

  Attachment #60848|0   |1
is obsolete||

--- Comment #5 from Lari Taskula  ---
Created attachment 69226
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69226=edit
Bug 18205: Add interface 'rest' for Koha::Logger

This patch adds an interface 'rest' for Koha::Logger.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Lari Taskula  changed:

   What|Removed |Added

  Attachment #60849|0   |1
is obsolete||

--- Comment #6 from Lari Taskula  ---
Created attachment 69227
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69227=edit
Bug 18205: Unit tests

To test:
1. prove t/db_dependent/Log.t

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Lari Taskula  changed:

   What|Removed |Added

  Attachment #60850|0   |1
is obsolete||

--- Comment #7 from Lari Taskula  ---
Created attachment 69228
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=69228=edit
Bug 18205: Mojo::Log with Koha::Logger::Mojo

This patch adds a new module, Koha::Logger::Mojo, based on
MojoX::Log::Log4perl.
This module allows us to use log4perl configurations with Mojolicious on the
freshly introduced 'rest' interface in log4perl.conf.

$c->app->log->warn will then use Log4perl as the underlying log mechanism and
those log events will point to the appropriate log file.

To test:
1. Install Perl dependency MojoX::Log::Log4perl
2. prove t/Koha/Logger/Mojo.t

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205
Bug 18205 depends on bug 18137, which changed state.

Bug 18137 Summary: REST API: Migrate from Mojolicious::Plugin::Swagger2 to 
Mojolicious::Plugin::OpenAPI
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18137

   What|Removed |Added

 Status|Pushed to Master|RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Josef Moravec  changed:

   What|Removed |Added

 CC||josef.mora...@gmail.com
 Status|Needs Signoff   |Patch doesn't apply

--- Comment #4 from Josef Moravec  ---
Does not apply on top of 18137...

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Lari Taskula  changed:

   What|Removed |Added

 Blocks||18206


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18206
[Bug 18206] REST API: Default exception handling
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Olli-Antti Kivilahti  changed:

   What|Removed |Added

 CC||olli-antti.kivila...@jns.fi
   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=18227

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Lari Taskula  changed:

   What|Removed |Added

 Depends on||18137


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18137
[Bug 18137] REST API: Migrate from Mojolicious::Plugin::Swagger2 to
Mojolicious::Plugin::OpenAPI
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

Lari Taskula  changed:

   What|Removed |Added

   Assignee|koha-b...@lists.koha-commun |lari.task...@jns.fi
   |ity.org |

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #3 from Lari Taskula  ---
Created attachment 60850
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60850=edit
Bug 18205: Mojo::Log with Koha::Logger::Mojo

This patch adds a new module, Koha::Logger::Mojo, based on
MojoX::Log::Log4perl.
This module allows us to use log4perl configurations with Mojolicious on the
freshly introduced 'rest' interface in log4perl.conf.

$c->app->log->warn will then use Log4perl as the underlying log mechanism and
those log events will point to the appropriate log file.

To test:
1. prove t/Koha/Logger/Mojo.t

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #2 from Lari Taskula  ---
Created attachment 60849
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60849=edit
Bug 18205: Unit tests

To test:
1. prove t/db_dependent/Log.t

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

[Koha-bugs] [Bug 18205] REST interface to Koha::Logger

[bugzilla-daemon]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18205

--- Comment #1 from Lari Taskula  ---
Created attachment 60848
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60848=edit
Bug 18205: Add interface 'rest' for Koha::Logger

This patch adds an interface 'rest' for Koha::Logger.

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/