profile
viewpoint
Matt Stauffer mattstauffer Tighten (@tightenco) Gainesville, FL http://mattstauffer.com/

mattstauffer/craftcms-prune 50

CraftCMS Prune Twig plugin

mattstauffer/alberts-avocado-toast 31

A simple example web site of some common geospatial and map embedding features

mattstauffer/dotfiles 20

.files, including ~/.osx — sensible hacker defaults for OS X

mattstauffer/craftcms-DownloadAssets 9

Craft CMS plugin to download Assets as a Zip

mattstauffer/happybrad 9

HappyBrad plugin for CraftCMS

mattstauffer/craftPluginDevHelpers 8

Helpers for Craft Plugin development

mattstauffer/awesome-helpers 4

Helper functions I find super-duper handy

push eventlaravel/valet

Beau Simensen

commit sha 46cdf1a155964485d12dcac1952d681456248b7d

Alias dev-master to 2.x-dev

view details

Matt Stauffer

commit sha 0d5d81c2578fac0ba3263c527fc9e2efc34b1c24

Merge pull request #950 from simensen/alias-master Alias dev-master to 2.x-dev

view details

push time in 7 days

PR merged laravel/valet

Alias dev-master to 2.x-dev

You all seem to be pretty good about updating cutting new releases and updating version numbers quickly, but I thought it would be nice to allow people to target @dev more easily (like ^2.10@dev or something).

+6 -1

4 comments

1 changed file

simensen

pr closed time in 7 days

pull request commentlaravel/valet

Alias dev-master to 2.x-dev

Thanks Beau!!

simensen

comment created time in 7 days

push eventtightenco/pamstack.org

Matt Stauffer

commit sha 32774666ba22ebc3a43339b503706df26808e3a7

Update README.md

view details

push time in 7 days

delete branch tightenco/onramp

delete branch : kdd/extract-translations

delete time in 7 days

pull request commenttightenco/onramp

Extract language translations from Blade to JS

Thanks @damiani!

damiani

comment created time in 7 days

push eventtightenco/onramp

Keith Damiani

commit sha 6400101b2a5725bd84f94edcdb348b4180d2990a

Remove translations from Blade

view details

Keith Damiani

commit sha c89f8c26bfda5f030d6fe612582ce3cd74fd93c7

Create translations.js file, import to app.js, update config

view details

Keith Damiani

commit sha 35e3f0148f473e2e30f20e142b12299b5908ee88

Update README

view details

Matt Stauffer

commit sha 86a2e3ab19296681a11fc956dfc7f4d352486fe9

Merge pull request #209 from tightenco/kdd/extract-translations Extract language translations from Blade to JS

view details

push time in 7 days

PR merged tightenco/onramp

Reviewers
Extract language translations from Blade to JS

This PR updates how language translations are accessed on the frontend. It removes the Blade directive that was used to include them in the markup for every page, and replaces it with a translations.js file that is loaded into Vue in app.js. (At the moment, the Vue frontend only uses 3 strings from that file.)

When changing or adding a translation, the translations.js file will need to be regenerated to pick up any changes. To regenerate translations.js, run the Artisan command:

php artisan export:messages-flat
+13 -29

0 comment

6 changed files

damiani

pr closed time in 7 days

push eventmattstauffer/recipes.mattstauffer.com

Matt Stauffer

commit sha c98f2cb781cb99b8e3be5399448602a3aabddff3

Add Martha Stewart pancakes

view details

push time in 7 days

push eventmattstauffer/matt-cooking

Matt Stauffer

commit sha 69297e72abd5a3a8bee36c01481f3673bb1901a7

Target blank source link

view details

push time in 7 days

push eventmattstauffer/matt-cooking

Matt Stauffer

commit sha 908ff580c3df792f8aab147f1b70a9c4996a33f4

Add sources to recipes

view details

push time in 7 days

push eventmattstauffer/matt-cooking

Matt Stauffer

commit sha 87c24011da859bbd7444ec6883f01cdc2e9cba31

Drop mailchimp

view details

Matt Stauffer

commit sha e2eac1d510f68792db40a1580670838692d10f7f

Build

view details

push time in 8 days

push eventmattstauffer/matt-cooking

Matt Stauffer

commit sha e348c20e23e90f7c68e77d25a370f834594169be

Rename everything blog to recipes, and add one

view details

push time in 8 days

create barnchmattstauffer/matt-cooking

branch : master

created branch time in 8 days

created repositorymattstauffer/matt-cooking

created time in 8 days

push eventtightenco/lambo

Matt Stauffer

commit sha b2e74a4939819e88a3d736328cbff642a95528b1

Filter out no-preset situations

view details

push time in 8 days

push eventtightenco/lambo

Matt Stauffer

commit sha 3d527823a3f4dfdc69492c1f819f401d2aecc00e

Delete todos

view details

push time in 8 days

issue openedtightenco/lambo

Write really simple docs for the beta

created time in 8 days

issue openedtightenco/lambo

Build tests for SetConfig

created time in 8 days

MemberEvent

issue openedtightenco/lambo

Re-work output formatters, etc. in presets to pass quiet flags so it doesn't throw a bunch of non-error errors (e.g. `Composer require`)

Our idea so far: Build a formatter style class whose responsibility it is to look at stderr output from each type of command and decide whether this is really an error

created time in 8 days

issue commenttightenco/collect

Error Class 'Tightenco\Collect\Support\Debug\Dumper' not found

Please provide more information. What version of Collect are you using? This is an old bug that I was under the impression was fixed; the only place I can reproduce it is the 5.5.* line. Are you using a 5.5.* version of Collect? If so, can you upgrade?

NanakiRus

comment created time in 12 days

push eventtightenco/tallstack

Liam Hammett

commit sha 6f31260665b6c69548e635196ec72df2633148a0

Add link to TALL preset

view details

Matt Stauffer

commit sha 8a94540bf792b69e513645cb40f138ae607049e6

Merge pull request #4 from imliam/patch-1 Add link to TALL preset

view details

push time in 12 days

PR merged tightenco/tallstack

Add link to TALL preset

This MR just adds a little info box that links to the TALL preset, so people browsing this site have something to get them rolling with the stack.

+22 -0

1 comment

1 changed file

imliam

pr closed time in 12 days

pull request commenttightenco/tallstack

Add link to TALL preset

@imliam I literally wrote a task in my journal this morning to add a link to it. 👌

imliam

comment created time in 12 days

pull request commentlaravel/framework

Job Batching

Oh my lord you don't know how excited I am for this!!

taylorotwell

comment created time in 14 days

push eventtightenco/builtwithjigsaw

Matt Stauffer

commit sha 7aa5764cc9c6b87ceb7d4d2a6c8db202d476ca6c

Add Pest PHP

view details

push time in 15 days

issue commenttightenco/lambo

Postgresql support

@Acciaiodigitale Hey, glad you found it!

I think adding postgres support is a very interesting idea. I'm open to it if someone has a way to PR it cleanly; if you'd like to do that you're welcome, but the main thing is trying to add it in a way that doesn't make the code or UI more complicated for the majority of users who are using MySQL.

In terms of the presets, we're currently working on building out an idea of presets in Lambo itsself, so there would be a "tall stack" preset you can apply when you use lambo that would trigger that preset you linked.

Thanks! :)

Acciaiodigitale

comment created time in 15 days

issue openedtightenco/onramp

Allow users to change their email address

created time in 17 days

issue commenttightenco/symposium

Make some sort of connection with CFPLand?

Karl and I talked earlier and I was saying how it seems like a lot of people are duplicating the effort of scraping all sorts of tech conference lists and normalizing them.

He pointed out that CallingAllPapers, with the lovely @heiglandreas, does this. I had forgotten until he said that, but we actually recently updated Symposium to pull our data from CallingAllPapers instead of joind.in.

I'm wondering--since Andreas is a friend, and his project is open source, would it be possible for us to imagine a suite of modifications that we could make to callingallpapers that Andreas likes that would make it function as the single source of truth for all of our platforms?

And, also, as always, can we at Symposium help any of y'all in any way? Also going to tag @timothep.

Just to note this publicly:

We've constantly seen folks try to enter the space of targeting speakers, CFP software, direct competition to Symposium, etc. and we just sort of go "hey, if you're passionate about this, go for it!" Symposium is a small labor of love and I know it's never going to be monetized. So I don't want to step on anyone's toes, but I'm also happy to be a part of the solution to everyone's needs, if such a thing exists, and I'd be happy to work with Andreas on updates to his project, if he's interested and we find it could be helpful to all of us.

mattstauffer

comment created time in 17 days

issue openedtightenco/symposium

Make some sort of connection with CFPLand?

https://www.cfpland.com/

@karllhughes

created time in 17 days

issue commenttightenco/gistlog

Breaks at whitespace not respected in Anchor elements

image

Answer from web.TA ^, for reference

dajare

comment created time in 18 days

issue commenttightenco/gistlog

Breaks at whitespace not respected in Anchor elements

@dajare oooops! Thanks, will take a look!

dajare

comment created time in 18 days

issue commenttightenco/jigsaw-blog-template

Where does $page->site_path come from? (rss.blade.php)

Hm. This is copied from my personal blog; in source/blog/feed.blade.atom, this is what mine looks like:

---
title: MattStauffer.com
site_path: blog
author: Matt Stauffer
---

@extends('_layouts.rss')

@section('entries')
    @foreach($posts as $entry)
        @include('_components.blog-as-rss-item')
    @endforeach
@endsection

That makes me think we modified it at some point in this template and didn't set that up correctly. But I'm kind of surprised that doesn't throw errors during generation. But yah, it seems like that's missing. Does that fix your problem? If so, we may need to fix the template.

Thanks!

raniesantos

comment created time in 18 days

push eventtightenco/builtwithjigsaw

Peter Brinck

commit sha 86775c111cae7ba8f98654e690c24e584d7e2466

Added Stanfeld website

view details

Matt Stauffer

commit sha 5e6a76a6fa4df11bd67fc97d7b5b14d746206e60

Merge pull request #94 from stanfeld/master Added Stanfeld website

view details

push time in 18 days

PR merged tightenco/builtwithjigsaw

Added Stanfeld website

Description here.

  • [x] This is adding a new web site to BuiltWithJigsaw
  • [ ] This is not a new web site, but something else

If this is adding a new web site:

  • [x] I have created a screenshot
  • [x] The screenshot is sized 380x210
  • [x] The screenshot is saved with PNG-8, not PNG-24, and I've run it through an image optimization tool like ImageOptim
  • [x] I saved the screenshot with the same filename as my site's .md file, but as a .png instead
  • [x] I saved the screenshot in the /source/assets/images/sites directory
  • [x] I built the site locally to ensure it looks right
+9 -0

1 comment

2 changed files

peterbrinck

pr closed time in 18 days

pull request commenttightenco/builtwithjigsaw

Added Stanfeld website

Nice work!

peterbrinck

comment created time in 18 days

issue closedlaravel/valet

Create a "diagnose" command

@drbyte regularly works hard to get useful output from folks as he's doing free customer support in the issues here.

I thought it might be nice if we could create a command or two that makes his job easier.

The first, I think, would be "diagnose", which gives him the output he always asks for on issues.

We could then create an issue template that requires the output of "diagnose".

I think the second one, later, would be more work, but could be more like a "doctor" command where it tries to actually find out what's wrong and possibly even tries to fix it. But right now, I think, it would just give information about the system.

Here's the commands he asked the last person I saw this with to run:

sw_vers
valet --version
cat ~/.config/valet/config.json
cat ~/.composer/composer.json
ls -al /etc/sudoers.d/
brew config
brew services list
brew list --versions | grep -E '(php|nginx|dnsmasq|mariadb|mysql|mailhog|openssl)(@\d\..*)?\s'
php -v
which -a php
php --ini
nginx -v
which -a php-fpm
sudo /usr/local/opt/php/sbin/php-fpm --test
sudo /usr/local/opt/php/sbin/php-fpm -y /usr/local/etc/php/7.4/php-fpm.conf --test
composer global diagnose

The sudo items are the only concern I have, so we might have to see whether those are optional or something.

closed time in 18 days

mattstauffer

push eventtightenco/tallstack

Dries Vints

commit sha d7e964113fcca0bdbebec2db4d89d2dcae584556

Update twitter link

view details

Matt Stauffer

commit sha dfe425331d63a2063b6c0756a3290ff8d89bf0a6

Merge pull request #3 from driesvints/patch-1 Update twitter link

view details

push time in 22 days

PR merged tightenco/tallstack

Update twitter link

Well this is awkward 👀

+1 -1

1 comment

1 changed file

driesvints

pr closed time in 22 days

pull request commenttightenco/tallstack

Update twitter link

Haaa!!! One day I’ll get the right one. Thanks!

driesvints

comment created time in 22 days

create barnchtightenco/laravel-scholarship

branch : master

created branch time in 22 days

created repositorytightenco/laravel-scholarship

created time in 22 days

created taglaravel/valet

tagv2.10.2

created time in 23 days

release laravel/valet

v2.10.2

released time in 23 days

push eventlaravel/valet

Matt Stauffer

commit sha ea45fcc80a2496550e6f6cdfa388afea113cd319

Drop unused directory from git export-ignore

view details

push time in 23 days

push eventlaravel/valet

Matt Stauffer

commit sha 7c357c59e2f8bbc7241675d2478137067b5d515c

Bump version

view details

push time in 23 days

pull request commentlaravel/valet

Stops interfering with favicon and robots for proxies

Thanks!

simensen

comment created time in 23 days

push eventlaravel/valet

Beau Simensen

commit sha 1959e95c5064ba4b214c5377a4486616b1a10952

Stops interfering with favicon and robots for proxies

view details

Matt Stauffer

commit sha c650aedfa8404cf8d3534908569145f5d3d15be0

Merge pull request #949 from simensen/proxies-no-block-favicon-and-robots Stops interfering with favicon and robots for proxies

view details

push time in 23 days

PR merged laravel/valet

Stops interfering with favicon and robots for proxies

The stub we used for normal usage as it relates to favicon.ico and robots.txt may not be appropriate for proxies. I'm seeing some weird behavior. I'm not even sure what the current configuration does, exactly, but if I remove them entirely Nginx forwards all requests to the proxy which is what I expected.

If someone has a better idea I'd be up for it. :) Otherwise, I think this probably fits better with the intent of the proxies.

+0 -18

0 comment

3 changed files

simensen

pr closed time in 23 days

push eventtightenco/builtwithjigsaw

Matt Stauffer

commit sha 75b8b7ecd56199513725b148ad582051e6123d9b

Add symlink to site images folder

view details

push time in 23 days

push eventtightenco/builtwithjigsaw

Matt Stauffer

commit sha 6421c153defcbfa0a471df5732c318284f4b0ea5

Flesh out readme instructions

view details

push time in 23 days

PR merged tightenco/builtwithjigsaw

Add freiberufler-tools.de

Description here.

  • [x] This is adding a new web site to BuiltWithJigsaw
  • [ ] This is not a new web site, but something else

If this is adding a new web site:

  • [x] I have created a screenshot
  • [x] The screenshot is sized 380x210
  • [x] The screenshot is saved with PNG-8, not PNG-24, and I've run it through an image optimization tool like ImageOptim
  • [x] I saved the screenshot with the same filename as my site's .md file, but as a .png instead
  • [x] I saved the screenshot in the /source/assets/images/sites directory
  • [x] I built the site locally to ensure it looks right
+9 -0

0 comment

2 changed files

tobiasdierich

pr closed time in 23 days

push eventtightenco/builtwithjigsaw

Tobias Dierich

commit sha 671d9860ba93b4ff58cd9d18296da7ee134a0c5a

Add freiberufler-tools.de

view details

Matt Stauffer

commit sha 8affdec93fe9b144595d91ca58e557a463e3d97c

Merge pull request #93 from tobiasdierich/master Add freiberufler-tools.de

view details

push time in 23 days

push eventtightenco/builtwithjigsaw

Adnan RIHAN

commit sha a689fe17077f1b9aa9f2e942127c98e453bc6c8a

Add coronavirus.tech.cg project

view details

Matt Stauffer

commit sha 55cc34beb4ccc1d929e8096972df3e837b3dca9a

Merge pull request #90 from Max13/corona-cg Add coronavirus.tech.cg project

view details

push time in 24 days

push eventtightenco/builtwithjigsaw

Matt Stauffer

commit sha 33c373eb85eab847d7d2d522a326ab93e62bed33

oOptimize adnan Rihan image and add optimization note to PR template

view details

push time in 24 days

push eventtightenco/builtwithjigsaw

Adnan RIHAN

commit sha 39c22a0785ab70605431b333c899e46130a47d20

Add Adnan RIHAN's Personal Website

view details

Matt Stauffer

commit sha e6585c2f07b340f987112aa52de3dee1c9c9c4d6

Merge pull request #89 from Max13/adnan-rihan-personal-website Add Adnan RIHAN's Personal Website

view details

push time in 24 days

push eventlaravel/valet

Matt Stauffer

commit sha aa09445d3a2820c787c501431536b3c5c1d127cf

Add issue template

view details

push time in 25 days

pull request commenttightenco/builtwithjigsaw

Update design with Ant's year-old changes

My quick notes upon review

  • I definitely like the new header
  • I like the cards idea for links, although I don’t totally love the spacing
  • wonder if he set it up that way so they wouldn’t break when they hit four lines
  • the “View all articles” button is broken, too.. it views more but not all.
  • we'd have to revert some changes I've made since he forked this.. like some of the accessibility and maybe some of the speed tweaks

Screenshots (this PR on the left, current site on the left): image

image

mattstauffer

comment created time in 25 days

PR opened tightenco/builtwithjigsaw

Update design with Ant's year-old changes

Test old Ant design that I just randomly found sitting on a branch.

+581 -120

0 comment

8 changed files

pr created time in 25 days

push eventtightenco/builtwithjigsaw

Matt Stauffer

commit sha f3ce13e59a535d630c4350db1466f578ca33f387

Tweak pull request template

view details

push time in 25 days

push eventtightenco/builtwithjigsaw

Matt Stauffer

commit sha c3da934ef5ac1c232a457cbb90ad03a1b50a25b6

Add pull request template

view details

push time in 25 days

PR merged tightenco/builtwithjigsaw

added easy recipes
+8 -0

0 comment

2 changed files

milon

pr closed time in 25 days

push eventtightenco/builtwithjigsaw

Nuruzzaman Milon

commit sha d3a6d78b7969eaa00fef640e2b7e6d108b187619

added easy recipes

view details

Matt Stauffer

commit sha 10e689a0a53a4d8415127cae08c58584a815c5bf

Resize easy recipes png

view details

Matt Stauffer

commit sha ea1ef75e5412438bc5fcd78ee8f3276452c7bc46

Merge branch 'pr/91'

view details

push time in 25 days

issue commenttightenco/nova-google-analytics

Does the Referrers List card work?

https://github.com/tightenco/nova-google-analytics/pull/10

Yah, this is a PR, so I was only relying on the PR author saying it worked for her. Any thoughts on what's wrong? I don't even have a great GA account to test it on right now.

dmyers

comment created time in a month

issue commenttightenco/nova-google-analytics

Does the Referrers List card work?

Hm. Let me take a look at whether I wrote that or whether someone else wrote it.

dmyers

comment created time in a month

pull request commenttightenco/nova-google-analytics

Fixed card classname capitalization in referrers list

Thanks!!

dmyers

comment created time in a month

push eventtightenco/nova-google-analytics

Derek Myers

commit sha 9be59666dd9829d7ad51c711615f73bd98444dad

Fixed card classname capitalization in referrers list

view details

Matt Stauffer

commit sha abdea395330298bf61f591132b36fe13df93206a

Merge pull request #16 from dmyers/patch-1 Fixed card classname capitalization in referrers list

view details

push time in a month

pull request commenttightenco/builtwithjigsaw

Add my personal site

Thanks! Looks great!

GenieTim

comment created time in a month

push eventtightenco/builtwithjigsaw

Tim Bernhard

commit sha e9621751600dc2ad2627b972475db020d8b84b9b

Create tim-bernhard-personal-site.md

view details

Matt Stauffer

commit sha ab78be526d5c56ed90999dcdab5c41362734eacd

Add screenshot of tim bernhard personal blog

view details

Matt Stauffer

commit sha 02c6dee3c2d42eb70c22cce4ca57ad431bbfd054

Merge branch 'GenieTim-patch-1'

view details

push time in a month

Pull request review commenttightenco/ziggy

Excellence

     "name": "ziggy",     "version": "0.5.0",     "description": "Generates a Blade directive exporting all of your named Laravel routes. Also provides a nice route() helper function in JavaScript.",+    "keywords": ["laravel", "routes", "ziggy"],     "main": "src/js/route.js",     "directories": {         "test": "tests/js"     },     "repository": {         "type": "git",-        "url": "git+https://github.com/tightenco/ziggy.git"+        "url": "https://github.com/tightenco/ziggy.git"     },-    "author": "Daniel Coulbourne",+    "authors": [+        {+            "name": "Daniel Coulbourne",+            "email": "daniel@tighten.co"+        },+        {+            "name": "Matt Stauffer",

I don't need to be listed on author on a project where I did very little! :) But I'm fine for Jake and Jacob to be added, if y'all write much code.

bakerkretzmar

comment created time in a month

created taglaravel/valet

tagv2.10.1

created time in a month

release laravel/valet

v2.10.1

released time in a month

push eventlaravel/valet

Matt Stauffer

commit sha fa090b0ab990e2a3e33c76c0e49f0ef8e65b4515

Bump version

view details

push time in a month

PR merged laravel/valet

Filter Launch* commands output in "diagnose" command

As discussed in https://github.com/laravel/valet/pull/938, this PR filters the Launch* commands to only show homebrew-related output.

Let me know if there's anything else!

+3 -3

1 comment

1 changed file

aryehraber

pr closed time in a month

push eventlaravel/valet

Aryeh Raber

commit sha 3be38193b532f5e6fcfab151681e7dd7f1968611

Filter Launch* commands using "| grep homebrew"

view details

Matt Stauffer

commit sha 570a7d905983e41bb886cf77d7560f1468ad460c

Merge pull request #942 from aryehraber/feat/diagnose-update Filter Launch* commands output in "diagnose" command

view details

push time in a month

pull request commentlaravel/valet

Filter Launch* commands output in "diagnose" command

@aryehraber I didn't even see this. Sorry! My email inbox is ... overwhelmed at the moment!

aryehraber

comment created time in a month

push eventtightenco/lambo

Matt Stauffer

commit sha bb935e49ea7620eb292dc249893414ffce870dab

Prep for context-specific help with presets with params - Passing the parameters to preset instance - Building proof of concept for differentiating composer packages based on user params - Extend basePreset to allow more customization

view details

push time in a month

push eventtightenco/lambo

Matt Stauffer

commit sha 122cb448e685d598e1640a0720fcdd743f308a3a

Build the stubs of a context-specific help command

view details

push time in a month

issue commenttightenco/lambo

Suggestion: New Help as One for All

Imagine this:

  1. Rename help-screen to help
  2. Allow for passing a parameter to help, with one of these options:
  • ./lambo help edit-config (command)
  • ./lambo help editor (parameter)
  • ./lambo help nova (preset)
muzafferdede

comment created time in a month

push eventtightenco/builtwithjigsaw

Matt Stauffer

commit sha c885a8d950307564c561265c4636642135068abf

Add @alexandersix personal site

view details

push time in a month

push eventtightenco/builtwithjigsaw

Matt Stauffer

commit sha d23a2ca70956d1d3632e8fd86ce428c1caa13e82

Add @alexandersix article

view details

push time in a month

release laravel/valet

v2.10.0

released time in a month

created taglaravel/valet

tagv2.10.0

created time in a month

push eventlaravel/valet

Matt Stauffer

commit sha 1dd6f301d55c122ae29b1b259b27f1adc96919aa

Bump version

view details

push time in a month

pull request commentlaravel/valet

Add proxy site-handling commands

Great work y'all!

drbyte

comment created time in a month

push eventlaravel/valet

Chris Brown

commit sha f056663159008531655298a6fa60b36e2d061530

Add proxy site-handling commands valet proxy domain host valet unproxy domain valet proxies (to list all recognized proxy site configs)

view details

Beau Simensen

commit sha 9f607ba99865de5638b68004d9a53e366757c708

Test the new Proxies features Updated the `proxies` method to return the URL + .tld as the rendered output wasn't exactly what one would expect. While not critical, it was not consistent. Refactored some of the Site class to aid in testing using a fake instead of relying on a ton of mocking. The Site fake has support for both using `tests/output` as well as named fixtures (`tests/fixtures/Proxies`). Testing for certificate I/O is pretty whack, but this is the best I could come up with that would still ensure the certificate stuff was getting called without actually requiring `sudo` to run phpunit. Replaced instances referring directly to `VALET_HOME_PATH` with calls to the new `valetHomePath()`. This method is taken over in the fake Site implementation so that everything runs the same using the fake Site without having to change other assumptions. Updated several "path" methods (and many of their usages) to take the "thing" you are looking for so you can either get the path to the type of "thing" you are looking for or the direct path to the specific "thing" you are looking for. Examples: ``` // ~/.config/valet/Nginx $site->nginxPath(); // ~/.config/valet/Nginx/some-site.com.test $site->nginxPath('some-site.com.test'); ``` Made some other tests related to the existence of `Sites` directory that resulted in updating the mocks for many of the other tests. All in all, it should make other aspects of the Site class handle things more gracefully if `Sites` doesn't exist.

view details

Beau Simensen

commit sha 1cd3d238043648ce1267288d5df784ba99ccd2ea

Update cli/Valet/Site.php Co-Authored-By: Matt Stauffer <mattstauffer@users.noreply.github.com>

view details

Chris Brown

commit sha 2ea3e58808a16069de1fafa65ba23b7c279e0cc8

Merge pull request #1 from simensen/valet-proxy-tests Test the Proxies features

view details

Beau Simensen

commit sha 02749e9e210fba308f28917f23c3e14985c2f08e

Add mocks to get tld

view details

Beau Simensen

commit sha 9848bc71b81cab792120b2083e842ffc611d0663

Forgot no proxies

view details

Chris Brown

commit sha 4043fd67c54d12c29204fdb81db05ab5ba6d167c

Merge pull request #2 from simensen/fix-valet-proxy-tests Add mocks to get tld

view details

Matt Stauffer

commit sha 73b16c08b0e84f41c622f5944a6897bddcce0c46

Merge pull request #913 from drbyte/valet-proxy Add proxy site-handling commands

view details

push time in a month

PR merged laravel/valet

Add proxy site-handling commands

Provides helpers to create an Nginx proxy site for the specified host. Useful for docker, vessel, mailhog etc, so that both Valet and those other services can co-exist on the same machine. (In the case of docker, proxying avoids the problem where Valet and Docker want to share port 80, which is not allowed.) (In the case of a service like mailhog, it already intends to run on a different port; but proxying allows this to be given a name that Valet recognizes and can serve quickly and easily.)

Adds 3 new commands: valet proxy [domain] [host] -- generates a proxy config file (see example below) valet unproxy [domain] -- removes a proxy config file valet proxies -- gives a list of proxies Valet is aware of

eg: valet proxy docker http://127.0.0.1:8080 creates a config file for docker.test, which proxies all traffic for https://docker.test through to http://127.0.0.1:8080

eg: valet proxy elasticsearch http://127.0.0.1:9200 creates a config file for elasticsearch.test, which proxies all traffic for https://elasticsearch.test accordingly.

eg: valet proxy mailhog http://localhost:8025 will proxy https://mailhog.test to the mailhog service running on localhost.

--

Credit for this comes from discussions with @simensen

--

Note: This implementation always "secures" the site, without needing to run valet secure on it.

--

Race condition (or "precedence"): A proxy config takes precedence because you can only have one config for a site. So, even if you have linked or secured a project to domain already, and run valet proxy with that domain, it will replace the link with the proxy config instead. To return to non-proxy for that domain, unproxy and then re-link and/or re-secure as you would normally for a non-proxy project. Or just don't re-use names unless truly important. :)

+977 -41

23 comments

8 changed files

drbyte

pr closed time in a month

pull request commentlaravel/valet

Add proxy site-handling commands

Re-reading now!

drbyte

comment created time in a month

pull request commenttightenco/onramp

Implement new front end design

Is learn.blade.php still in use?

Hm I'd lean toward no? But let's see lol

TammyTee

comment created time in a month

issue commentlaravel/valet

404 for files in public/ (LaravelValetDriver)

@stidges Great catch there. That's a fascinating issue... I'm curious how we can get rid of the error logs but also make it work

skollro

comment created time in a month

pull request commentlaravel/valet

Add "diagnose" command

I totally agree it's not private! But if it's not necessary I'd love to limit it :)

@aryehraber if we're grepping, no need for a separate command.

Thanks both of you!

aryehraber

comment created time in a month

Pull request review commentdrbyte/valet

Test the Proxies features

 function proxies()         })->reject(function ($host, $site) {             // If proxy host is null, it may be just a normal SSL stub, or something else; either way we exclude it from the list             return $host === '(other)';-        })->map(function ($host, $site) use ($certs) {+        })->map(function ($host, $site) use ($certs, $tld) {             $secured = $certs->has($site);-            $url = ($secured ? 'https': 'http').'://'.$site;

This seems like a reasonable change, but was it really just not showing TLD in the previous incarnation? Or do we have something that shows differently depending on the context?

simensen

comment created time in a month

Pull request review commentdrbyte/valet

Test the Proxies features

 function getSiteConfigFileContents($site, $dir = null)      * @param string $path      * @return \Illuminate\Support\Collection      */-    function getCertificates($path)+    function getCertificates($path = null)     {+        $path = is_null($path) ? $this->certificatesPath() : $path;
        $path = $path ?: $this->certificatesPath();
simensen

comment created time in a month

Pull request review commentdrbyte/valet

Test the Proxies features

 function getProxyHostForSite($site, $configContents = null)         return $host;     } -    function getSiteConfigFileContents($site, $dir = null)

Just to double check, nowhere else in the app was using this method and passing the dir?

simensen

comment created time in a month

more