Merge pull request #9036 from joshcooper/delete_pson

(PUP-11783) Delete vendored pson

Author: mhashizume

.rubocop_todo.yml

@@ -155,7 +155,6 @@ Layout/EmptyLinesAroundAccessModifier:
 Layout/EmptyLinesAroundArguments:
   Exclude:
     - 'lib/puppet/defaults.rb'
-    - 'lib/puppet/external/pson/common.rb'
     - 'lib/puppet/parser/functions/sprintf.rb'
     - 'lib/puppet/settings.rb'
     - 'lib/puppet/type.rb'
@@ -601,15 +600,9 @@ Lint/ElseLayout:
   Exclude:
     - 'lib/puppet/resource.rb'
 
-# Configuration parameters: AllowComments.
-Lint/EmptyConditionalBody:
-  Exclude:
-    - 'lib/puppet/external/pson/pure/parser.rb'
-
 # Configuration parameters: AllowComments.
 Lint/EmptyWhen:
   Exclude:
-    - 'lib/puppet/external/pson/pure/parser.rb'
     - 'lib/puppet/indirector/hiera.rb'
     - 'lib/puppet/pops/evaluator/evaluator_impl.rb'
     - 'lib/puppet/pops/lookup/data_dig_function_provider.rb'
@@ -853,7 +846,6 @@ Naming/ClassAndModuleCamelCase:
 
 Naming/ConstantName:
   Exclude:
-    - 'lib/puppet/external/pson/common.rb'
     - 'lib/puppet/graph/relationship_graph.rb'
     - 'lib/puppet/indirector/hiera.rb'
     - 'lib/puppet/provider/package/sun.rb'
@@ -909,7 +901,6 @@ Naming/VariableName:
     - 'ext/windows/service/daemon.rb'
     - 'lib/puppet/agent.rb'
     - 'lib/puppet/application/describe.rb'
-    - 'lib/puppet/external/pson/common.rb'
     - 'lib/puppet/pops/lookup/hiera_config.rb'
     - 'lib/puppet/pops/validation/checker4_0.rb'
     - 'lib/puppet/provider/package/pip.rb'
@@ -1019,11 +1010,6 @@ Style/CaseEquality:
 Style/CaseLikeIf:
   Enabled: false
 
-# This cop supports safe auto-correction (--auto-correct).
-Style/CharacterLiteral:
-  Exclude:
-    - 'lib/puppet/external/pson/pure/parser.rb'
-
 # This cop supports safe auto-correction (--auto-correct).
 # Configuration parameters: EnforcedStyle.
 # SupportedStyles: nested, compact
@@ -1121,8 +1107,6 @@ Style/DoubleNegation:
   Exclude:
     - 'lib/puppet/application/lookup.rb'
     - 'lib/puppet/confine/boolean.rb'
-    - 'lib/puppet/external/pson/pure/generator.rb'
-    - 'lib/puppet/external/pson/pure/parser.rb'
     - 'lib/puppet/http/service/compiler.rb'
     - 'lib/puppet/parser/functions/fqdn_rand.rb'
     - 'lib/puppet/pops/evaluator/evaluator_impl.rb'
@@ -1146,8 +1130,6 @@ Style/EmptyBlockParameter:
 # This cop supports safe auto-correction (--auto-correct).
 Style/EmptyCaseCondition:
   Exclude:
-    - 'lib/puppet/external/pson/common.rb'
-    - 'lib/puppet/external/pson/pure/parser.rb'
     - 'lib/puppet/pops/lookup/hiera_config.rb'
     - 'lib/puppet/pops/serialization/serializer.rb'
     - 'lib/puppet/pops/types/type_calculator.rb'
@@ -1220,8 +1202,6 @@ Style/ExplicitBlockArgument:
 # SupportedStyles: each, for
 Style/For:
   Exclude:
-    - 'lib/puppet/external/pson/common.rb'
-    - 'lib/puppet/external/pson/pure/generator.rb'
     - 'lib/puppet/file_system/uniquefile.rb'
     - 'lib/puppet/module_tool/contents_description.rb'
     - 'lib/puppet/util/rdoc/generators/puppet_generator.rb'
@@ -1294,8 +1274,6 @@ Style/HashTransformValues:
 # This cop supports safe auto-correction (--auto-correct).
 Style/IdenticalConditionalBranches:
   Exclude:
-    - 'lib/puppet/external/pson/common.rb'
-    - 'lib/puppet/external/pson/pure/generator.rb'
     - 'lib/puppet/functions/filter.rb'
     - 'lib/puppet/network/http_pool.rb'
     - 'lib/puppet/node.rb'
@@ -1848,7 +1826,6 @@ Style/SignalException:
 # Configuration parameters: AllowIfMethodIsEmpty.
 Style/SingleLineMethods:
   Exclude:
-    - 'lib/puppet/external/pson/pure/generator.rb'
     - 'lib/puppet/ffi/windows/api_types.rb'
     - 'lib/puppet/file_system/memory_file.rb'
     - 'lib/puppet/graph/simple_graph.rb'
@@ -2007,7 +1984,6 @@ Style/TrivialAccessors:
 Style/UnlessElse:
   Exclude:
     - 'lib/puppet/application/lookup.rb'
-    - 'lib/puppet/external/pson/pure/parser.rb'
     - 'lib/puppet/pops/evaluator/evaluator_impl.rb'
     - 'lib/puppet/pops/types/type_mismatch_describer.rb'
     - 'lib/puppet/provider/package/aix.rb'
@@ -2026,7 +2002,6 @@ Style/UnpackFirst:
 # This cop supports safe auto-correction (--auto-correct).
 Style/VariableInterpolation:
   Exclude:
-    - 'lib/puppet/external/pson/pure/parser.rb'
     - 'lib/puppet/util/command_line/trollop.rb'
     - 'lib/puppet/util/rdoc/generators/puppet_generator.rb'
 

api/docs/http_api_index.md

@@ -66,9 +66,9 @@ available and how to interact with it.
 These services are all directly used by the Puppet agent application, in order
 to manage the configuration of a node.
 
-These endpoints accept payload formats formatted as JSON or PSON (MIME types of
-`application/json` and `text/pson`, respectively) except for `File Content` and
-`File Bucket File` which always use `application/octet-stream`.
+These endpoints accept payload formats formatted as JSON (MIME type
+`application/json`) except for `File Content` and `File Bucket File` which
+always use `application/octet-stream`.
 
 * [Facts](./http_facts.md)
 * [Catalog](./http_catalog.md)
@@ -133,10 +133,10 @@ The certificate authority (CA) API contains all of the endpoints supporting Pupp
 Serialization Formats
 ---------------------
 
-Puppet sends messages using several different serialization formats. Not all
+Puppet sends messages using `JSON` or as binary data. Not all
 REST services support all of the formats.
 
 * [JSON](https://tools.ietf.org/html/rfc7159)
-* [PSON](./pson.md)
+* [PSON](./pson.md) was supported in earlier versions of Puppet, but is no longer for performance reasons.
 
 `YAML` was supported in earlier versions of Puppet, but is no longer for security reasons.

api/docs/http_catalog.md

@@ -17,7 +17,7 @@ POST, GET
 
 ### Supported Response Formats
 
-`application/json`, `text/pson`
+`application/json`
 
 ### Notes
 
@@ -34,8 +34,8 @@ The examples below use the POST method.
 Four parameters should be provided to the POST or GET:
 
 - `environment`: the environment name.
-- `facts_format`: must be `application/json` or `pson`.
-- `facts`: serialized JSON or PSON of the facts hash. Since facts can contain `&`, which
+- `facts_format`: must be `application/json`.
+- `facts`: serialized JSON of the facts hash. Since facts can contain `&`, which
   is also the HTTP query parameter delimiter, facts are doubly-escaped.
 - `transaction_uuid`: a transaction uuid identifying the entire transaction (shows up in the report as well).
 

api/docs/http_facts.md

@@ -16,7 +16,7 @@ PUT
 
 ### Supported Format(s)
 
-`application/json`, `text/pson`
+`application/json`
 
 ### Parameters
 

api/docs/http_file_metadata.md

@@ -19,7 +19,7 @@ The endpoint path includes a `:mount` which can be one of the following types:
 * `pluginfacts` --- a highly magical mount point which merges the `facts.d` directory of every module together. Used for syncing external facts; not intended for general consumption. Per-module sub-paths can not be specified.
 * `tasks/<MODULE>` --- a semi-magical mount point which allows access to files in the `tasks` subdirectory of `<MODULE>` --- see the [the docs on file serving](https://puppet.com/docs/puppet/latest/file_serving.html).
 
-Note: PSON responses in the examples below are pretty-printed for readability.
+Note: JSON responses in the examples below are pretty-printed for readability.
 
 Find
 ----
@@ -34,7 +34,7 @@ GET
 
 ### Supported Response Formats
 
-`application/json`, `text/pson`
+`application/json`
 
 ### Parameters
 
@@ -54,7 +54,7 @@ Optional parameters to GET:
     GET /puppet/v3/file_metadata/modules/example/just_a_file.txt?environment=env
 
     HTTP/1.1 200 OK
-    Content-Type: text/pson
+    Content-Type: application/json
 
     {
         "checksum": {
@@ -76,7 +76,7 @@ Optional parameters to GET:
     GET /puppet/v3/file_metadata/modules/example/subdirectory?environment=env
 
     HTTP/1.1 200 OK
-    Content-Type: text/pson
+    Content-Type: application/json
 
     {
         "checksum": {
@@ -98,7 +98,7 @@ Optional parameters to GET:
     GET /puppet/v3/file_metadata/modules/example/link_to_file.txt?environment=env&source_permissions=ignore
 
     HTTP/1.1 200 OK
-    Content-Type: text/pson
+    Content-Type: application/json
 
     {
         "checksum": {
@@ -136,7 +136,7 @@ GET
 
 ### Supported Response Formats
 
-`application/json`, `text/pson`
+`application/json`
 
 ### Parameters
 
@@ -156,7 +156,7 @@ GET
     GET /puppet/v3/file_metadatas/modules/example?environment=env&recurse=yes
 
     HTTP 200 OK
-    Content-Type: text/pson
+    Content-Type: application/json
 
     [
         {
@@ -236,7 +236,7 @@ GET
     GET /puppet/v3/file_metadatas/modules/example?environment=env&recurse=true&ignore=sub*&links=manage
 
     HTTP 200 OK
-    Content-Type: text/pson
+    Content-Type: application/json
 
     [
         {
@@ -285,7 +285,7 @@ GET
 
 #### Search ignoring "sub*" and links = follow
 
-This example is identical to the above example, except for the links parameter. The resulting PSON, then,
+This example is identical to the above example, except for the links parameter. The resulting JSON, then,
 is identical to the above example, except for:
 
 * the "links" field is set to "follow" rather than "manage" in all metadata objects
@@ -298,7 +298,7 @@ is identical to the above example, except for:
 GET /puppet/v3/file_metadatas/modules/example?environment=env&recurse=true&ignore=sub*&links=follow
 
 HTTP 200 OK
-Content-Type: text/pson
+Content-Type: application/json
 
 [
     {

api/docs/http_node.md

@@ -22,7 +22,7 @@ GET
 
 ### Supported Response Formats
 
-`application/json`, `text/pson`
+`application/json`
 
 ### Parameters
 
@@ -38,7 +38,7 @@ environment, which might differ from what the client believes is its current env
 ### Examples
 
     > GET /puppet/v3/node/mycertname?environment=production&transaction_uuid=aff261a2-1a34-4647-8c20-ff662ec11c4c&configured_environment=production HTTP/1.1
-    > Accept: application/json, text/pson
+    > Accept: application/json
 
     < HTTP/1.1 200 OK
     < Content-Type: application/json

api/docs/http_report.md

@@ -21,7 +21,7 @@ PUT
 
 ### Supported Format(s)
 
-`application/json`, `text/pson`
+`application/json`
 
 ### Parameters