Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retrieve base host URL without path in error processor #244

Closed
witrin opened this issue Feb 21, 2013 · 1 comment
Closed

Retrieve base host URL without path in error processor #244

witrin opened this issue Feb 21, 2013 · 1 comment

Comments

@witrin
Copy link

witrin commented Feb 21, 2013

Hi,

you use $_SERVER['HTTP_HOST'], which may contain the port number also, in Error_Processor::getHostUrl(). On a Magento instance using the port 8080 it produces in my case the host URL localhost:8080:8080.

Best regards,
Artus

magento-team added a commit that referenced this issue Mar 7, 2014
* Cache:
  * Implemented depersonalization of private content generation
  * Implemented content invalidation
  * Added Edge Side Includes (ESI) support
  * Added a built-in caching application
* GitHub requests:
  * [#454](#454) -- Allow to specify list of IPs in a body on maintenance.flag which will be granted access even if the flag is on
  * [#204](#204) -- Mage_ImportExport: Exporting configurable products ignores multiple configurable options
  * [#418](#418) -- Echo vs print
  * [#419](#419) -- Some translation keys are not correct.
  * [#244](#244) -- Retrieve base host URL without path in error processor
  * [#411](#411) -- Missed column 'payment_method' of table 'sales_flat_quote_address'
  * [#284](#284) -- Fix for Issue #278 (Import -> Stores with large amount of Configurable Products)
* Fixed bugs:
  * Fixed an issue where Mage_Eav_Model_Entity_Type::fetchNewIncrementId() did not rollback on exception
  * Fixed an issue where a category containing more than 1000 products could not be saved
  * Fixed inappropriate error messages displayed during installation when required extensions were not installed
  * Fixed synopsis of the install.php script
  * Fixed an issue where the schedule of recurring payments was not displayed in the shopping cart
* Modularity improvements:
  * Introduced the OfflinePayments module - a saparate module for offline payment methods
  * Added the ability to enable/disable the Paypal module
  * Moved the framework part of the Locale functionality from the Core module to library
  * The Locale logic was split among appropriate classes in library, according to their responsibilities
  * Removed the deprecated DHL functionality
  * Introduced the OfflineShipping module for offline shipping carrier functionality: Flatrate, Tablerate, Freeshipping, Pickup
  * Introduced a separate module for the DHL shipping carrier
  * Introduced a separate module for the Fedex shipping carrier
  * Introduced a separate module for the UPS shipping carrier
  * Introduced a separate module for the USPS shipping carrier
* Framework Improvements:
  * Added the ability to intercept internal public calls
  * Added the ability to access public interface of the intercepted object
  * Added a static integrity test for plugin interface validation
  * Added support for both class addressing approaches in DI: with and without slash ("\") at the beginning of a class name
* Customer Service usage:
  * Refactored the Customer module blocks and controllers to use customer service layer
* Security:
  * Introduced the ability to hash a password with a random salt of default length (32 chars) by the encryption library
  * Utilized a random salt of default length for admin users, and frontend customers
@verklov
Copy link
Contributor

verklov commented Mar 8, 2014

@witrin, the team has fixed the issue you reported. The code has been released in dev68.

@verklov verklov closed this as completed Mar 8, 2014
magento-team pushed a commit that referenced this issue Dec 16, 2015
[DRAGONS] Sprint 77 - Bugs and MultiLocationInventory
magento-engcom-team added a commit that referenced this issue Nov 15, 2018
 - Merge Pull Request magento/graphql-ce#244 from VoronoyAlexandr/graphql-ce:graphql_abstract_missed_phpdoc_argument
 - Merged commits:
   1. dc51f3e
   2. 6062a1e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants