Paths

Table of Contentst

Differential D253

Revert "Set correct default app.key configuration setting"
ClosedPublic
Actions

Authored by dereckson on Jan 22 2016, 04:09.

Details

Reviewers

dereckson

Maniphest Tasks

T691: Revert APP_KEY to a dummy non 32 character value

Commits

Unknown Object (Diffusion Commit)
rNOTIF292935a0c8d4: Revert "Set correct default app.key configuration setting"

Summary

This reverts commit 53de6354a0b38b229c091ff1fdf44f48eff7de7e.

T690 shows a case where it would be very welcome the application
loudly fails with an encryption key issue exception.

Test Plan

Ensure CI tools (Jenkins jobs, insight.sensiolabs.com)
can create a .env file.

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 593.Jan 22 2016, 04:09

dereckson retitled this revision from to Revert "Set correct default app.key configuration setting".

dereckson updated this object.

dereckson edited the test plan for this revision. (Show Details)

dereckson added a reviewer: dereckson.

dereckson added a subscriber: Sandlayth.

Build started: https://ci.nasqueron.org/job/test-notifications-phab/46/ (console: https://ci.nasqueron.org/job/test-notifications-phab/46/console)

dereckson accepted this revision.Jan 22 2016, 04:10

dereckson edited edge metadata.

This revision is now accepted and ready to land.Jan 22 2016, 04:10

Build has FAILED

Link to build: https://ci.nasqueron.org/job/test-notifications-phab/46/
See console output for more information: https://ci.nasqueron.org/job/test-notifications-phab/46/console

Perfect, every test making requests failed with RuntimeException: No supported encrypter found. The cipher and / or key length are invalid..

Closed by commit rNOTIF292935a0c8d4: Revert "Set correct default app.key configuration setting" (authored by dereckson, committed by dereckson). · Explain WhyJan 22 2016, 04:16

This revision was automatically updated to reflect the committed changes.

dereckson mentioned this in D255: Allow phpunit tests to run without .env file.Jan 22 2016, 04:48

dereckson added a task: T691: Revert APP_KEY to a dummy non 32 character value.Jan 22 2016, 05:01

dereckson mentioned this in rNOTIFbc6b78c70675: Allow phpunit tests to run without .env file.Jan 22 2016, 05:03

dereckson mentioned this in Q4: A PHP application source code has been updated, but all behaves like before. (Answer 4).Jan 24 2016, 07:45

Revision Contents
Changeset List

Path

Size

config/

app.php

2 lines

Diff 594

View Options

Revert "Set correct default app.key configuration setting"ClosedPublicActions