Skip to content

Flask january release.#1

Open
sasakalaba wants to merge 18 commits intomasterfrom
flask_january_release
Open

Flask january release.#1
sasakalaba wants to merge 18 commits intomasterfrom
flask_january_release

Conversation

@sasakalaba
Copy link
Copy Markdown
Owner

@sasakalaba sasakalaba commented Feb 28, 2017

  • Added ConfigurationError from stormpath-flask.
  • Added MoveAPIKeyToClientAPIKeyStrategy (in specifications).
  • Added MoveSettingsToConfigStrategy (custom).
  • Added validation to EnrichIntegrationFromRemoteConfigStrategy.
  • Added OverridingStrategiesTest (simulating overriding strategies in
    ConfigLoader).

Comment thread stormpath_config/errors.py Outdated

class ConfigurationError(Exception):
"""
This exception is raised if a user has misconfigured Flask-Stormpath.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix docstring

Comment thread stormpath_config/loader.py Outdated
for strategy in self.post_processing_strategies:
config = strategy.process(config)

config = _extend_dict(config, config.pop('stormpath', {}))
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove

from .load_apikey_from_config import LoadAPIKeyFromConfigStrategy


class MoveStormpathSettingsToStormpathConfig(object):
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strategy in class name


# Check the format of application information.
if stormpath_key == 'application':
if 'https://api.stormpath.com/v1/applications' in value:
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sasakalaba sasakalaba changed the title Flask february release. Flask january release. Feb 28, 2017
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

Successfully merging this pull request may close these issues.

2 participants