[pve-devel] [PATCH 0/9] get rid of PVE::REST

Fabian Grünbichler f.gruenbichler at proxmox.com
Tue Jan 10 15:58:55 CET 2017


On Tue, Jan 10, 2017 at 11:55:17AM +0100, Dietmar Maurer wrote:
> I want to split out HTTPServer into a separate package. This is much
> easiere if we remove the PVE::REST class, which does not really add
> any value.
> 

see feedback on individual patches - only minor nitpicks (which could
also be fixed in follow-up patches)

patch #8 seems to be unrelated?

> 
> Dietmar Maurer (9):
>   HTTPServer.pm: add cookie handling methods
>   HTTPServer.pm: add auth_handler
>   HTTPServer.pm: allow to set baseuri when creating the class
>   HTTPServer.pm: add rest_handler method
>   HTTPServer.pm: add missing 'use' statements
>   PVE/Service/pveproxy.pm: avoid using PVE::REST
>   PVE/API2/Formatter/HTML.pm: avoid use of PVE::REST
>   bin/Makefile: export PERLLIB=.. so that doc generator works correctly
>   remove obsolete PVE::REST class
> 
>  PVE/API2/Formatter/HTML.pm |   5 +-
>  PVE/HTTPServer.pm          | 214 ++++++++++++++++++++++++++++++++++++++++++---
>  PVE/Makefile               |   1 -
>  PVE/REST.pm                | 203 ------------------------------------------
>  PVE/Service/pveproxy.pm    |   2 +-
>  bin/Makefile               |   2 +
>  6 files changed, 206 insertions(+), 221 deletions(-)
>  delete mode 100644 PVE/REST.pm
> 
> -- 
> 2.1.4
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel




More information about the pve-devel mailing list