aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| | * | | | | | | | | Routing is a named concept nowDavid Heinemeier Hansson2018-09-195-11/+11
| | | | | | | | | | |
| | * | | | | | | | | Test routing runs through a job kicked off by the inbound emailDavid Heinemeier Hansson2018-09-191-0/+7
| | | | | | | | | | |
| | * | | | | | | | | Temp routing directly to named controllerDavid Heinemeier Hansson2018-09-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Need the singleton to live somewhere proper.
| | * | | | | | | | | Everyone needs job testingDavid Heinemeier Hansson2018-09-193-5/+2
| | | | | | | | | | |
| | * | | | | | | | | Ensure raw email is created firstDavid Heinemeier Hansson2018-09-191-5/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise jobs hanging off the InboundEmail won't be able to access the Active Storage data (as it won't have been uploaded yet).
| | * | | | | | | | | Simpler class definitionDavid Heinemeier Hansson2018-09-191-6/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't actually remember why we've used the explicit module wrapper before?
| | * | | | | | | | | Add easy way to get logging information inline for test failure diagnosisDavid Heinemeier Hansson2018-09-191-0/+4
| | | | | | | | | | |
| | * | | | | | | | | Attach a concrete router to the root mailbox and use itDavid Heinemeier Hansson2018-09-196-14/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't think this is how it's going to stay. Doesn't feel like the right place for it.
| | * | | | | | | | | Included in test helper nowDavid Heinemeier Hansson2018-09-191-2/+0
| | | | | | | | | | |
| | * | | | | | | | | Add incineration by defaultDavid Heinemeier Hansson2018-09-195-0/+76
| | | | | | | | | | |
| | * | | | | | | | | Follow db table conventionDavid Heinemeier Hansson2018-09-191-1/+1
| | | | | | | | | | |
| | * | | | | | | | | Tmp filesDavid Heinemeier Hansson2018-09-191-0/+1
| | | | | | | | | | |
| | * | | | | | | | | Make debugger available in testingDavid Heinemeier Hansson2018-09-191-0/+2
| | | | | | | | | | |
| | * | | | | | | | | Missing suffixDavid Heinemeier Hansson2018-09-191-0/+0
| | | | | | | | | | |
| | * | | | | | | | | Only deliver pending emails to the mailroomDavid Heinemeier Hansson2018-09-183-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Makes it easier to test without triggering this behavior.
| | * | | | | | | | | Inherit from ActiveJob::Base rather than ApplicationJobDavid Heinemeier Hansson2018-09-182-7/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't want to trigger app-specific behavior here.
| | * | | | | | | | | Prefix queue name like we do routesDavid Heinemeier Hansson2018-09-181-1/+1
| | | | | | | | | | |
| | * | | | | | | | | Make a note for tying inbound email to exceptionDavid Heinemeier Hansson2018-09-181-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Then InboundEmail doesn't need to serialize or track the exception that went with it. The arrow will point the other way.
| | * | | | | | | | | Add callbacksDavid Heinemeier Hansson2018-09-183-2/+60
| | | | | | | | | | |
| | * | | | | | | | | Extract mailbox test suites into feature-specific slicesDavid Heinemeier Hansson2018-09-182-36/+34
| | | | | | | | | | |
| | * | | | | | | | | More scalable name!David Heinemeier Hansson2018-09-181-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There'll probably be other concerns we need to do around the processing.
| | * | | | | | | | | Proper orderDavid Heinemeier Hansson2018-09-181-1/+1
| | | | | | | | | | |
| | * | | | | | | | | Process inbound emails with state and exceptionsDavid Heinemeier Hansson2018-09-182-1/+51
| | | | | | | | | | |
| | * | | | | | | | | Use rails scope by defaultDavid Heinemeier Hansson2018-09-181-1/+1
| | | | | | | | | | |
| | * | | | | | | | | Add test for unsupported content typeDavid Heinemeier Hansson2018-09-182-0/+9
| | | | | | | | | | |
| | * | | | | | | | | Remember to add access protectionDavid Heinemeier Hansson2018-09-181-0/+1
| | | | | | | | | | |
| | * | | | | | | | | Use a test helper to create fixtures rather than rely on them being predefinedDavid Heinemeier Hansson2018-09-183-23/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Too much hassle to manage all the steps compared to just pointing to an .eml fixture and having it setup for you.
| | * | | | | | | | | Add basic, unauthenticated inbound emails controllerDavid Heinemeier Hansson2018-09-173-0/+40
| | | | | | | | | | |
| | * | | | | | | | | Assert the intended email was processedDavid Heinemeier Hansson2018-09-171-2/+2
| | | | | | | | | | |
| | * | | | | | | | | No byebug history plzDavid Heinemeier Hansson2018-09-171-0/+1
| | | | | | | | | | |
| | * | | | | | | | | Keep the riff raff outDavid Heinemeier Hansson2018-09-171-0/+3
| | | | | | | | | | |
| | * | | | | | | | | Save this for a little laterDavid Heinemeier Hansson2018-09-171-1/+1
| | | | | | | | | | |
| | * | | | | | | | | First end-to-end rickety testDavid Heinemeier Hansson2018-09-178-24/+656
| | | | | | | | | | |
| | * | | | | | | | | WIP: RouterDavid Heinemeier Hansson2018-09-173-0/+36
| | | | | | | | | | |
| | * | | | | | | | | Add dummy app for testingDavid Heinemeier Hansson2018-09-1778-2/+7400
| | | | | | | | | | |
| | * | | | | | | | | Wait on this for laterDavid Heinemeier Hansson2018-09-171-3/+0
| | | | | | | | | | |
| | * | | | | | | | | Full name for the jobDavid Heinemeier Hansson2018-09-171-1/+1
| | | | | | | | | | |
| | * | | | | | | | | TOC orderingDavid Heinemeier Hansson2018-09-171-1/+3
| | | | | | | | | | |
| | * | | | | | | | | Plural table nameDavid Heinemeier Hansson2018-09-171-1/+1
| | | | | | | | | | |
| | * | | | | | | | | Base -> MailboxDavid Heinemeier Hansson2018-09-173-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Won't need Base and can separate routing from execution 👍
| | * | | | | | | | | Action Mailbox -> Action MailroomDavid Heinemeier Hansson2018-09-1712-36/+36
| | | | | | | | | | |
| | * | | | | | | | | Add inbound emailDavid Heinemeier Hansson2018-09-173-0/+37
| | | | | | | | | | |
| | * | | | | | | | | Copypasta is delicious!David Heinemeier Hansson2018-09-171-1/+1
| | | | | | | | | | |
| | * | | | | | | | | SkeletonDavid Heinemeier Hansson2018-09-1711-0/+160
| | / / / / / / / /
* | | | | | | | | | Merge pull request #34791 from y-yagi/fixes_34787Yuji Yaginuma2018-12-261-0/+221
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Generate doc of methods provided by form builders [ci skip]
| * | | | | | | | | | Generate doc of methods provided by form builders [ci skip]yuuji.yaginuma2018-12-261-0/+221
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Because method arguments are different in the methods provided by form helpers and form builders, I think these are necessary to prevent confusion. Fixes #34787
* / / / / / / / / / Specify `hosts` in bug report template.yuuji.yaginuma2018-12-251-0/+1
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Host authentication introduced by #33145 allows only "0.0.0.0/0", "::/0" and "localhost" in the development environment. https://github.com/rails/rails/blob/4298df00ae6219b9b5b7c40f281d4fa4d66f4383/railties/lib/rails/application/configuration.rb#L33 But `rack-test` use `example.org` for default host. https://github.com/rack-test/rack-test/blob/6c07bf53adf4a4082e206561f5b9b43142fdc821/lib/rack/test.rb#L13 Therefore, if `hosts` is not specified, host authentication rejects the request. In Travis CI, this problem does not occur because `test` is specified by default in `RAILS_ENV` and `RACK_ENV`. https://docs.travis-ci.com/user/environment-variables/#default-environment-variables If user actually use it, env may not always be specified. Explicitly specify hosts in the file so that it works in any environment.
* | | | | | | | | Merge pull request #34781 from kossnocorp/patch-1Kasper Timm Hansen2018-12-241-2/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Make Rails Facebook-free
| * | | | | | | | | Make Rails Facebook-freeSasha Koss2018-12-241-2/+2
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | If Basecamp is a Facebook-free business, then Rails should be Facebook-free framework.
* | | | | | | | | Add missing require for `IPAddr`yuuji.yaginuma2018-12-241-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ref: https://travis-ci.org/rails/rails/jobs/469956825#L1694
generated by cgit v1.2.3 (git 2.25.1) at 2025-03-22 23:50:07 +0000