-#
-# Catlyst::Action::SerializeBase.pm
-# Created by: Adam Jacob, Marchex, <adam@hjksolutions.com>
-#
-# $Id$
-
package Catalyst::Action::SerializeBase;
-use strict;
-use warnings;
+use Moose;
+use namespace::autoclean;
-use base 'Catalyst::Action';
+extends 'Catalyst::Action';
use Module::Pluggable::Object;
use Catalyst::Request::REST;
use Catalyst::Utils ();
-sub new {
- my $class = shift;
- my $config = shift;
- Catalyst::Request::REST->_insert_self_into(
- Catalyst::Utils::class2appclass($config->{class})
- );
- return $class->SUPER::new($config, @_);
-}
+after BUILDARGS => sub {
+ my $class = shift;
+ my $config = shift;
+ Catalyst::Request::REST->_insert_self_into( $config->{class} );
+};
-__PACKAGE__->mk_accessors(qw(_serialize_plugins _loaded_plugins));
+has [qw(_serialize_plugins _loaded_plugins)] => ( is => 'rw' );
sub _load_content_plugins {
my $self = shift;
my $sclass = $search_path . "::";
my $sarg;
my $map;
+ my $compliance_mode;
+ my $default;
my $config;
if ( exists $controller->{'serialize'} ) {
- $c->log->info("Using deprecated configuration for Catalyst::Action::REST!");
- $c->log->info("Please see perldoc Catalyst::Action::REST for the update guide");
+ $c->log->info("Catalyst::Action::REST - deprecated use of 'serialize' for configuration.");
+ $c->log->info("Please see 'CONFIGURATION' in Catalyst::Controller::REST.");
$config = $controller->{'serialize'};
# if they're using the deprecated config, they may be expecting a
# default mapping too.
$config = $controller;
}
$map = $config->{'map'};
+ $default = $config->{'default'} if $config->{'default'};
+
+ # If we're in RFC 7231 compliance mode we need to determine if we're
+ # serializing or deserializing, then set the request object to
+ # look at the appropriate set of supported content types.
+ $compliance_mode = $config->{'compliance_mode'};
+ if($compliance_mode) {
+ my $serialize_mode = (split '::', $search_path)[-1];
+ if($serialize_mode eq 'Deserialize') {
+ # Tell the request object to only look at the Content-Type header
+ $c->request->set_content_type_only();
+
+ # If we're in compliance mode and doing deserializing we want
+ # to use the allowed content types for deserializing, not the
+ # serializer map
+ $map = $config->{'deserialize_map'};
+ $default = $config->{'deserialize_default'} if $config->{'deserialize_default'};
+ } elsif($serialize_mode eq 'Serialize') {
+ # Tell the request object to only look at the Accept header
+ $c->request->set_accept_only();
+ }
+ }
# pick preferred content type
my @accepted_types; # priority order, best first
# then content types requested by caller
push @accepted_types, @{ $c->request->accepted_content_types };
# then the default
- push @accepted_types, $config->{'default'} if $config->{'default'};
+ push @accepted_types, $default if $default;
# pick the best match that we have a serializer mapping for
my ($content_type) = grep { $map->{$_} } @accepted_types;
- return $self->_unsupported_media_type($c, $content_type)
+ return $self->unsupported_media_type($c, $content_type)
if not $content_type;
# carp about old text/x-json
$sclass .= $mc;
#}
if ( !grep( /^$sclass$/, @{ $self->_serialize_plugins } ) ) {
- return $self->_unsupported_media_type($c, $content_type);
+ return $self->unsupported_media_type($c, $content_type);
}
} else {
- return $self->_unsupported_media_type($c, $content_type);
+ return $self->unsupported_media_type($c, $content_type);
}
unless ( exists( $self->_loaded_plugins->{$sclass} ) ) {
my $load_class = $sclass;
if ($@) {
$c->log->error(
"Error loading $sclass for " . $content_type . ": $!" );
- return $self->_unsupported_media_type($c, $content_type);
+ return $self->unsupported_media_type($c, $content_type);
} else {
$self->_loaded_plugins->{$sclass} = 1;
}
}
if ($search_path eq "Catalyst::Action::Serialize") {
- if ($content_type) {
- $c->response->header( 'Vary' => 'Content-Type' );
- } elsif ($c->request->accept_only) {
- $c->response->header( 'Vary' => 'Accept' );
+ unless( $c->response->header( 'Vary' ) ) {
+ if ($content_type) {
+ $c->response->header( 'Vary' => 'Content-Type' );
+ } elsif ($c->request->accept_only) {
+ $c->response->header( 'Vary' => 'Accept' );
+ }
}
$c->response->content_type($content_type);
}
return $sclass, $sarg, $content_type;
}
-sub _unsupported_media_type {
+sub unsupported_media_type {
my ( $self, $c, $content_type ) = @_;
$c->res->content_type('text/plain');
$c->res->status(415);
return undef;
}
-sub _serialize_bad_request {
+sub serialize_bad_request {
my ( $self, $c, $content_type, $error ) = @_;
$c->res->content_type('text/plain');
$c->res->status(400);
return undef;
}
+__PACKAGE__->meta->make_immutable;
+
1;
=head1 NAME
-B<Catalyst::Action::SerializeBase>
-
-Base class for Catalyst::Action::Serialize and Catlayst::Action::Deserialize.
+Catalyst::Action::SerializeBase - Base class for Catalyst::Action::Serialize and Catlayst::Action::Deserialize.
=head1 DESCRIPTION
L<Catalyst::Action::Serialize>, L<Catalyst::Action::Deserialize>,
L<Catalyst::Controller::REST>,
-=head1 AUTHOR
-
-Adam Jacob <adam@stalecoffee.org>, with lots of help from mst and jrockway.
+=head1 AUTHORS
-Marchex, Inc. paid me while I developed this module. (http://www.marchex.com)
+See L<Catalyst::Action::REST> for authors.
=head1 LICENSE
You may distribute this code under the same terms as Perl itself.
=cut
-