Jira (PUP-2630) Server-set global variables like $::environment get overwritten by client facts

[Henrik Lindberg (JIRA)]

Title: Message Title

Henrik Lindberg updated an issue   Puppet / PUP-2630 Server-set global variables like $::environment get overwritten by client facts Change By: Henrik Lindberg REPORTED PROBLEM --- As per the docs (http://docs.puppetlabs.com/puppet/latest/reference/lang_facts_and_builtin_vars.html#variables-set-by-the-puppet-master) the puppet master sets the global $::environment variable to contain the name of the node's environment. However, if a node provides a fact with the name 'environment' that fact's value overrides the server-set environment. The same happens with other server-set global variables, like $::servername and $::serverip. As a result, modules can't reliably use these variables for whatever their intended purpose is. If this is the intended behavior, then the docs should clarify this and discourage use of the server-set variables, because a misbehaving node can override them. SOLUTION --- Add a function that returns the server facts (e.g. {{server_facts()}} since that makes it possible for module authors to depend on puppet 4.1.0 and get expected values without any opt-in. This is preferred over setting yet another global variable name. *  Separate the logic in node (see comment from Andy Parker below with the details) to make it possible to retrieve the server side facts later *  Add a function {{server_facts()}} that returns a hash of the server side facts  (that were set aside in the node) * If there are no server side facts (puppet apply case), an empty hash is returned * Issue a warning when a fact overwrites a server side fact (but perform the overwrite since not doing so breaks API) * Decide on real resolution of this issue: (Ping [~eric.sorenson] ) ** Possibly  i.e. possibly  deprecate setting server facts as variables in top scope?    So  (Add deprecation ticket ,  add removal ticket  for  4  PUP 5 . 1. 0  - make the separation in the node logic to enable the function to get correct data, but keep the variable overwrites .  Also add a warning when there is an overwrite of a serverfact by a fact. 0) QA --- risk: high probability: medium severity: high (hard to debug, somewhat of a security issue, can spoof environment) test layer: acceptance Add Comment   This message was sent by Atlassian JIRA (v6.3.15#6346-sha1:dbc023d) -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to puppet-bugs+unsubscr...@googlegroups.com. To post to this group, send email to puppet-bugs@googlegroups.com. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/d/optout.