« first day (4373 days earlier)      last day (571 days later) » 

12:06 AM
@OlleHärstedt You're the 3rd person (after JRL and myself) that seem to remember that added restriction being mentioned but can find no evidence of it.
 
 
2 hours later…
2:22 AM
 
2:34 AM
youtube.com/watch?v=ze4xcmBFvaE (cry + strings + perc., vocals etc.)
 
2:46 AM
youtube.com/… (classico)
 
 
5 hours later…
7:50 AM
Hi guys, some XML Dsig experts here? If I sign my SAMLResponse and I want to sign Assertion and the whole message. Would I first need to sign the Assertion and than pass content (with signed elements) to sign the whole message, or do I need to do theses steps separately?
 
8:01 AM
@Crell haha funny
So we need yet another keyword? purereadonly? :D
really-i-mean-it-readnonly
 
8:33 AM
@Crell tbf that's due to the const changes which allow to assign it objects
Making it more like a const pointer
And that threw me for a loop when I realised that
I sometimes wonder if the PHP 4 object model wasn't on some degree more sensible in that it was CoW AFAIK (never wrote PHP 4 code so who knows)
 
It wasn't copy on write, it was copy on reassign.
 
@Girgias Yeah, like const in JS
 
8:59 AM
@Derick Isn't that kinda the same or am I missing something here?
 
$a = $b; (copy on assign)
$a = $b; $b->foo = 42; (copy on write)
CoW is usually used when something is changed only
 
Ahhhh
 
9:22 AM
@TimWolla docs.github.com/en/actions/using-jobs/… "For each object in the include list, the key:value pairs in the object will be added to each of the matrix combinations if none of the key:value pairs overwrite any of the original matrix values." That is so dubious. Any clue how to disable that?
I think we're getting hit by this because some jobs that don't modify configuration_parameters or run_tests_parameters don't specify it. And then GitHub actions essentially merges these jobs...
 
9:36 AM
@IluTov Is this about the nightly job?
This is also the first time I'm seeing the documentation, I've not ever included keys in the include bit that aren't also part of the main matrix.
 
@TimWolla Yeah. I'll try adding empty defaults to the main matrix. Hopefully that'll work.
 
10:38 AM
@cmb Should I just go and merge?
 
cmb
@Girgias github.com/php/php-src/pull/9680 is okay to merge, but needs to go into PHP-8.2 too (I'll check why no test failed). I'm coming to github.com/php/php-src/pull/9638 in a moment.
 
@cmb I merged it in 8.2 :)
Okay thanks, just saw you commented on some session related tickets and was wondering if the PR just slipped your mind
 
10:59 AM
Are you all also receiving '[turkdevops/php-src] Run failed at startup: Push - master (32d101b)' email notifications whenever you push to php/php-src?
It's a little annoying that GitHub sends me such notifications for a repository I didn't actually interact with, just because they messed up their GHA config.
 
cmb
Yeah, get these notifications too. I've decided to filter them out.
 
same here, yes
 
@TimWolla Yes I had them, but I went and forcefully unwatch the repo
So I don't get them anymore :)
 
11:33 AM
@Girgias Ah, ignoring the repository is a good hint. I just did that, thanks.
 
cmb
+1
 
 
2 hours later…
1:25 PM
So got round to writing some words about FCI/FCC for the internals book, please review: github.com/phpinternalsbook/PHP-Internals-Book/pull/135
 
1:56 PM
... and now I need a burger today. youtube.com/watch?v=hKy1xI05kcw
 
 
2 hours later…
3:53 PM
@Girgias Thanks for writing this. I had to figure out much of that on my own a couple years ago, and many of the nuances have escaped me now.
 
@Trowski No problem, I've been learning this again this week :|
Got like 20 tabs open cross-checking different files on heap.space lol
 
@TimWolla github.com/php/php-src/commit/… That did the trick. A bit ugly but whatever.
 
@IluTov I don't find that ugly at all. At least it's explicit about what parameters are theoretically available.
 
 
3 hours later…
 
3 hours later…
10:36 PM
I have a bunch of controllers in webapp, each serves a form, when a form is submitted the next controller (page/route) is loaded. I do this with match() i.e. controllerA => controllerB, controllerB => controllerC etc. So when controllerB is submitted it routes to controllerC. (the names are example and not really with A, B, C)
I currently have the controller match() code in a Service class but this feels wrong for routing, is it ok or is there a better place for it to live?
 
JRL
10:49 PM
why not use one of the router packages on composer?
 
is that not overkill? I'm just matching one controller name to another and using the matched one to return a redirect response (Symfony)
 

« first day (4373 days earlier)      last day (571 days later) »