PWA

I have checked the following:

[X] I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add
Make Bruno a Progressive Web App (PWA)
Related

https://www.pwabuilder.com/
https://web.dev/explore/progressive-web-apps

File system is entirely accessible from the web api nowaways!

https://developer.mozilla.org/en-US/docs/Web/API/File_System_API
https://developer.mozilla.org/en-US/docs/Web/API/File_API

That would be a significant undertaking. What would be the benefits of doing that?

I personally would love to use Bruno on iPhone and iPad.
Not sure if Bruno is responsive enough for iPhone yet (I don't think so) but it could be something to work on.

That would be a significant undertaking. What would be the benefits of doing that?

Cross-platform even on arcane Linux or BSD, for example, even video games consoles
Also no need for electron anymore

+1
It would be great to run bruno as a web-app so it can run remotely or even locally inside a docker container (https://github.com/usebruno/bruno/issues/1415), so no need install any desktop app, reducing the effort for adopting it in a team.


Update Contentful.java

https://github.com/contentful/contentful.java/releases/tag/java-sdk-7.3.0
which now supports fallback locales

Currently the module accepts only the 1st locale and drops the rest.

Fallback locales were added to Contentful platform as configuration. They can be set for the project in settings.
They can't, however, be requested on API call as of this release.
Only thing that was added in this skd release for fallback locales was the possibility to get them from fetched models.


[Question] Possibility of videos with motion-corresponding bone outputs similar to RAC in Lab4D

Hi @gengshan-y .
Thank you for the great work.
In Lab4D, is it possible to produce videos similar to RAC and their corresponding bone outputs?
Looking forward to your response.

Hi. There is a ppr branch that implements mesh rendering with bones (instructions here). We plan to merge it once more testing is done.
It would also be possible to port the pyrender wrapper and mesh renderer code directly with some effort.

Thank you for your response.
I'll give the ppr branch a try!
Looking forward to the merge!


Updated compatibility to Laravel 5.5

Walked trough the upgrade guide, no changes had to be made to the code according to the guide.

Could you add package auto-discovery support as well?

Good Idea, didn't think about that. I will update the PR tomorrow as I don't have the time now.

Tested with a personal project and the package auto-discovery works :smile:

Maybe the README also should get an update? (auto-discovery) 😄

Yeah, but I don't know if @HipsterJazzbo wants to do it himself or if it should be included with the PR.

Now really fixed the auto-discovery. The facade wasn't correctly implemented before.

Looking forward to this.

Pleeeeease!! =)

Looks good :)


[lua] update to 5.4.0

Fix #12344
Update lua to the latest version 5.4.0
All features have passed with the following triplets:

x64-windows
x86-windows
x64-windows-static

/azp run

Because even Lua minor version bumps break compatibility and some downstream libraries thus lock to specific versions of Lua, is it vcpkg policy to still keep one port up-to-date over some side-by-side approach?

/azp run

Thanks :D !


Support for authentication & authorization webhooks

It's nice to have ability to configure authentication and authorization webhooks during setup. It can't be simply achieved by adding options to apiserver since it also requires extra configuration file that should be available in apiserver POD.
It also possible related with this.

Thanks @bacher09. Do you think you'd be able to create a proposal or PR for this?

Actually I would like to discuss implementation first before making any PR.
I think that easiest way to do this, is create directory with custom data that will be copied to '/etc/kubernetes' on master nodes. So webhook config can be placed in such directory and it will be copied there before kubernetes services are started. Also, this design is generic enough to be useful for other use cases, since many options of 'kube-apiserver' requires file. For example, if someone is using authorization via oidc they can provide '--oidc-ca-file' in same way. To make this design really reusable for some other cases, it will probably require more than one directory. Maybe something like 6 of them. Since, in some cases it also useful to provide custom manifests, for example authentication webhook service can be self-hosted and provided by custom manifest. Also, users need to specify where this files would be placed, on master node, on slave or on both of them.
So, what do you think about such design. Is it ok ?

@alexbrand pls take a look https://github.com/apprenda/kismatic/pull/1118


Improve Livy performance for long execution queries

Fix for https://github.com/rstudio/sparklyr/issues/1538, fixed livy.session.command.timeout and support for  livy.session.command.interval to control max polling while waiting for command response.

Thanks Javier.  Yes, that fix is what I want. There should be a reasonably small maximum value for the time to sleep.

Sounds good.
BTW. My initial fix had a typo, so if you synced to this branch, sync again. I will merge once test succeed.

Hi @javierluraschi, when do you think the next release (including this improvement) will be ready?


targetHost 'localhost' is not an ADC

ISSUE TYPE

Bug Report

AS3 BUILD/ VERSION

3.5.1-5

BIGIP VERSION

Sys::Version
Main Package
  Product     BIG-IP
  Version     <IP_ADDRESS>
  Build       0.0.4
  Edition     Point Release 4
  Date        Wed Jan 16 18:04:36 PST 2019

SUMMARY

Declaration returns a 404 with a message: "targetHost 'localhost' is not an ADC"
The declaration does not use the targetHost attribute

STEPS TO REPRODUCE

sensitive information removed and replaced with xxxx
{
    "class": "AS3",
    "action": "deploy",
    "persist": true,
    "declaration": {
        "class": "ADC",
        "schemaVersion": "3.0.0",
        "id": "example-declaration-01",
        "label": "Sample 1",
        "remark": "Simple HTTP application with round robin pool",
        "www": {
            "class": "Tenant",
            "Application_1": {
                "class": "Application",
                "template": "http",
                "serviceMain": {
                    "class": "Service_HTTP",
                    "virtualAddresses": [
                        "<IP_ADDRESS>"
                    ],
                    "pool": "web_pool",
                    "policyWAF": {
                        "bigip": "/Common/blocking.misc.prod"
                    }
                },
                "web_pool": {
                    "class": "Pool",
                    "monitors": [
                        "http"
                    ],
                    "members": [{
                        "servicePort": 80,
                        "addressDiscovery": "azure",
                        "updateInterval": 10,
                        "tagKey": "name",
                        "tagValue": "demo-dev-app-nic",
                        "addressRealm": "private",
                        "resourceGroup": "demo-dev-rg",
                        "subscriptionId": "xxxx",
                        "directoryId": "xxxx",
                        "applicationId": "xxxx",
                        "apiAccessKey": "xxxx",
                        "credentialUpdate": false

                    }]
                }
            }
        }
    }
}

EXPECTED RESULTS

Successful deployment of declaration

ACTUAL RESULTS

On some BIG-IPs the declaration works and on others, I receive
the error stated above.  BIG-IPs in this environment are deployed
via Terraform and the A&O toolchain is installed using the Ansible
role.  This issue occurs in ~30% of my deployments.

restarting restnoded does not resolve this problem.
AS3 is installed and seems to be working:
GET https://{{F5_SERVER}}/mgmt/shared/appsvcs/info returns:
{
"version": "3.5.1",
"release": "5",
"schemaCurrent": "3.5.0",
"schemaMinimum": "3.0.0"
}

f5admin@(demo)(cfg-sync Standalone)(Active)(/Common)(tmos.sys.provision)# list
sys provision afm { }
sys provision am { }
sys provision apm { }
sys provision asm {
level nominal
}
sys provision avr { }
sys provision dos { }
sys provision fps { }
sys provision gtm { }
sys provision ilx { }
sys provision lc { }
sys provision ltm {
level nominal
}
sys provision pem { }
sys provision swg { }
sys provision urldb { }

To decide if the target host (default localhost) is a BIG-IP, AS3 probes for a valid response to an iControl REST query for /mgmt/tm/sys/version.  Take a look in the logs (you may have to set log-level to debug) for more info about what is going wrong.

This was also reported by another source, and is in our internal product backlog as AT-749.

We're also hitting this issue in a PAYG instance in Azure that has been running for months and this just all of a sudden happened.

This issue was resolved in the 3.16.0 release.


FE 2.6 show some information,is it right?

i can not  figure out what is going .

They are probably deleted nodes in 2.6.
I will update. please wait few days.

I fixed Fill & EmptyDrawingHnalde nodes.
getClosest*** nodes are renamed GetClosest***, so sorry.


#187 - Add force flag to prevent unexpected overwrite of files.

This PR adds a -Force flag to ConvertTo-Bicep.
If the output .bicep file already exists in target directory we will no longer default to overwriting it, but instead throw a non terminating error to the console.
As this is considered a breaking change it should not be included in the 1.x release tree, but in 2.x

ConvertTo-Bicep has been changed to work with the new BicepNet nested module. All 2.0 related changes should be done to the Version2.0 branch.

updated and fixed changes so it merges in to 2.0 branch instead.


Please add sticky behaviour to elements (nav) bars

Hi, looking at your documentation:

      Logo
      menu

Sass

Components

JavaScript

Test 1

Test 2

Disabled Tab

Test 4

I thought it would be useful for responsive and mobile-first principle to have a sticky   ##behaviour of the navbar, as well as elements in lists.

P.S. How to add directly a feature requests in your Project "Trello" view ?
Or only possible to add it as issue and then you select them ?

Have you tried out our pushpin component? I think it achieves what you are looking for:
https://materializecss.com/pushpin


Allow HTTP protocol selection

allow protocol selection via AttributeMap and SdkHttpConfigurationOption.PROTOCOL
fixes #225

Thanks for the PR @jtjeferreira. I am currently on vacation. I will have a look next week.

What do you think of adding an integration test, which is using Http2 and also adding a paragraph in the Readme how Http2 can be enabled?

I tried that but only localstack supports http/2...

Do you know which AWS services support Http2?

I could not find any documentation except https://docs.aws.amazon.com/sdk-for-java/latest/developer-guide/http2.html, and its mention to kinesis using http2. However if I do curl -v --http2 https://kinesis.us-east-1.amazonaws.com it uses http/1.1. If I do curl -v --http2 https://lambda.us-east-1.amazonaws.com it uses http/2 🤷

Do you understand what is the issue with the CI?

Hi @matsluni, I added some tests and improved the README

@matsluni ready for re-review

I just found this: https://doc.akka.io/docs/akka-http/current/client-side/http2.html. I am wondering now, if our approach here is not really the thing we expect it to do?

I just found this: doc.akka.io/docs/akka-http/current/client-side/http2.html.

You mean any particular section?

I am wondering now, if our approach here is not really the thing we expect it to do?

Honestly, I am not sure either what to expect. I think that setting the protocol in the akka.http.scaladsl.model.HttpRequest is having an effect. Otherwise, some of the tests (the S3 and SQS tests), where I use http/2 specifically, would not be failing.
Also since this is an opt-in setting and we don't change anything for existing clients, I think this is a safe PR...
I started exploring the internals of this library because I am having some issues with random connection closures, but I think  https://github.com/matsluni/aws-spi-akka-http/pull/228 is much more promising in that regard. So if you are unsure about this PR, we can close it...

Hi @jtjeferreira, I tried the approach I mentioned above in this branch: protocol_other_approach. What do you think about this?

I don't think we should use the "Connection-Level Client-Side API" (i.e Http().connectionTo) because we will be opening a new connection for every request.
Besides the use of the Source.queue is flawed because it is not failing requests properly and that's why the tests don't finish. We could remove the queue and use:
  def singleRequest(connection: Flow[HttpRequest, HttpResponse, Any], bufferSize: Int = 100): HttpRequest => Future[HttpResponse] = {
    req => {
      Source.single(req)
        .via(connection)
        .runWith(Sink.head)
    }
  }

The local tests using Http/2 are now do not finish anymore and therefore I commented them out.

The local tests are failing because we are not setting the port. We need to do
Http()
  .connectionTo(request.request().host())
  .toPort(request.request().port())
  .http2()

Actually, I would expect that the local tests using Http/2 are not working, as the local test services are not exposing Http/2 functionality (TLS etc).

You can have HTTP/2 without TLS, and akka supports this with http2WithPriorKnowledge() instead of using http2(). If we modify the code to be
    if (protocol == HttpProtocols.`HTTP/2.0`) {
      val baseConnection = Http().connectionTo(request.request().host()).toPort(request.request().port())
      val connection = request.request().protocol() match {
        case "http" => baseConnection.http2WithPriorKnowledge()
        case "https" => baseConnection.http2()
        case _ => throw new IllegalArgumentException("Unsupported protocol")
      }
      val dispatch = singleRequest(connection)
      runner.run(
        () => dispatch(akkaHttpRequest),
        request.responseHandler()
      )
    }

all test are green except:

com.github.matsluni.akkahttpspi.sqs.TestSQS because we are sending an unencryped http/2 request, the server replies with HTTP/1.1 501 Not Implemented, but akka-http fails with java.lang.IllegalStateException: truncated data in ByteStringParser
com.github.matsluni.akkahttpspi.s3.TestS3 again because we are sending an unencryped http/2 request, the server replies with HTTP/1.1 426 Upgrade Required, but akka-http fails again with java.lang.IllegalStateException: truncated data in ByteStringParser

After writing the above comment, and looking at the debug logs of tests I realized that my approach of just setting the HttpRequest.protocol to HTTP/2.0 is just changing the HTTP version sent in the HTTP plaintext payload, and no features of HTTP like compression are multiplexing are being used.
my approach
[DEBUG] [10/31/2023 12:45:37.429] [aws-akka-http-akka.actor.default-dispatcher-7] [akka.stream.Log(akka://aws-akka-http/system/Materializers/StreamSupervisor-3)] [client-plain-text ToNet  ] Element: SendBytes  ByteString(272 bytes) first + last 100:
 47 45 54 20 2F 20 48 54 54 50 2F 32 2E 30 0D 0A  | GET / HTTP/2.0..
 48 6F 73 74 3A 20 6C 6F 63 61 6C 68 6F 73 74 3A  | Host: localhost:
 34 39 35 35 35 0D 0A 61 6D 7A 2D 73 64 6B 2D 69  | 49555..amz-sdk-i
 6E 76 6F 63 61 74 69 6F 6E 2D 69 64 3A 20 31 38  | nvocation-id: 18
 34 62 31 64 32 35 2D 37 63 64 33 2D 30 32 61 33  | 4b1d25-7cd3-02a3
 2D 35 37 31 30 2D 65 37 63 36 31 63 34 61 39 34  | -5710-e7c61c4a94
 63 31 0D 0A                                      | c1..
                     ... [72 bytes omitted] ...
 6E 4A 44 4B 5F 36 34 2D 42 69 74 5F 53 65 72 76  | nJDK_64-Bit_Serv
 65 72 5F 56 4D 2F 31 31 2E 30 2E 32 30 2B 30 20  | er_VM/11.0.20+0
 4A 61 76 61 2F 31 31 2E 30 2E 32 30 20 73 63 61  | Java/11.0.20 sca
 6C 61 2F 32 2E 31 33 2E 31 30 20 76 65 6E 64 6F  | la/2.13.10 vendo
 72 2F 48 6F 6D 65 62 72 65 77 20 69 6F 2F 61 73  | r/Homebrew io/as
 79 6E 63 20 68 74 74 70 2F 55 4E 4B 4E 4F 57 4E  | ync http/UNKNOWN
 0D 0A 0D 0A                                      | ....
[DEBUG] [10/31/2023 12:45:37.433] [aws-akka-http-akka.actor.default-dispatcher-7] [akka.stream.Log(akka://aws-akka-http/system/Materializers/StreamSupervisor-3)] [client-plain-text FromNet] Element: SessionBytes  ByteString(208 bytes) first + last 100:
 48 54 54 50 2F 31 2E 31 20 34 32 36 20 55 70 67  | HTTP/1.1 426 Upg
 72 61 64 65 20 52 65 71 75 69 72 65 64 0D 0A 43  | rade Required..C
 6F 6E 74 65 6E 74 2D 54 79 70 65 3A 20 74 65 78  | ontent-Type: tex
 74 2F 68 74 6D 6C 3B 63 68 61 72 73 65 74 3D 69  | t/html;charset=i
 73 6F 2D 38 38 35 39 2D 31 0D 0A 43 6F 6E 74 65  | so-8859-1..Conte
 6E 74 2D 4C 65 6E 67 74 68 3A 20 35 39 0D 0A 43  | nt-Length: 59..C
 6F 6E 6E 65                                      | onne
                     ... [8 bytes omitted] ...
 6C 6F 73 65 0D 0A 53 65 72 76 65 72 3A 20 4A 65  | lose..Server: Je
 74 74 79 28 39 2E 34 2E 35 31 2E 76 32 30 32 33  | tty(9.4.51.v2023
 30 32 31 37 29 0D 0A 0D 0A 3C 68 31 3E 42 61 64  | 0217)....
Bad
 20 4D 65 73 73 61 67 65 20 34 32 36 3C 2F 68 31  |  Message 426

reason: Up
 67 72 61 64 65 20 52 65 71 75 69 72 65 64 3C 2F  | grade Required</
 70 72 65 3E                                      | pre>

your approach
[DEBUG] [10/31/2023 12:48:07.584] [aws-akka-http-akka.actor.default-dispatcher-9] [akka.stream.Log(akka://aws-akka-http/system/Materializers/StreamSupervisor-3)] [client-plain-text ToNet  ] Element: SendBytes  ByteString(24 bytes)
 50 52 49 20 2A 20 48 54 54 50 2F 32 2E 30 0D 0A  | PRI * HTTP/2.0..
 0D 0A 53 4D 0D 0A 0D 0A                          | ..SM....
[DEBUG] [10/31/2023 12:48:07.584] [aws-akka-http-akka.actor.default-dispatcher-9] [Http2ClientDemux(akka://aws-akka-http)] Changing state from WaitingForNetworkToSendControlFrames to Idle
[DEBUG] [10/31/2023 12:48:07.589] [aws-akka-http-akka.actor.default-dispatcher-9] [akka.stream.Log(akka://aws-akka-http/system/Materializers/StreamSupervisor-3)] [client-plain-text ToNet  ] Element: SendBytes  ByteString(21 bytes)
 00 00 0C 04 00 00 00 00 00 00 03 00 00 01 00 00  | ................
 02 00 00 00 00                                   | .....
[DEBUG] [10/31/2023 12:48:07.589] [aws-akka-http-akka.actor.default-dispatcher-9] [Http2ClientDemux(akka://aws-akka-http)] Changing state from Idle to WaitingForData
[DEBUG] [10/31/2023 12:48:07.590] [aws-akka-http-akka.actor.default-dispatcher-9] [akka.stream.Log(akka://aws-akka-http/system/Materializers/StreamSupervisor-3)] [client-plain-text ToNet  ] Element: SendBytes  ByteString(189 bytes)
 00 00 B4 01 05 00 00 00 01 82 86 41 8B A0 E4 1D  | ...........A....
 13 9D 09 B8 D3 EE 00 5F 84 5C 01 30 40 8F 1D 3E  |<EMAIL_ADDRESS> D6 44 9D 56 35 5D CE 41 A4 C7 A9 63 49 99 0C 82  | .D.V5].A...cI...
 29 0B 1B 8B 3E 57 82 B4 AE B8 F2 C3 0B 84 58 45  | )...>W........XE
 78 81 19 48 37 07 27 40 8A 1D 3E D6 44 9D 56 B0  |<EMAIL_ADDRESS> A9 B3 D7 83 03 00 63 7A E4 1F 84 2C 89 3A AD D0  | ......cz...,.:..
 7D C6 C0 97 08 57 69 4D 03 24 5A B7 45 F8 C0 59  | }....WiM.$Z.E..Y
 5C 4B 85 4D 55 96 AC B7 F3 44 E3 4B 5D 32 62 DC  | \K.MU....D.K]2b.
 5B 3B 96 C8 B8 E8 60 21 5C 0B 88 1F EC 0A 65 1F  | [;....`!\.....e.
 71 B0 10 AE 05 C4 05 10 41 D0 36 04 B8 59 5C 20  | q.......A.6..Y\
 53 B9 6A 90 F6 31 8C F4 96 3B 0B E1 43 1D 81 A3  | S.j..1...;..C...
 D5 44 52 74 A6 B6 38 69 CD A7 57 2D 3F           | .DRt..8i..W-?
[DEBUG] [10/31/2023 12:48:07.595] [aws-akka-http-akka.actor.default-dispatcher-9] [akka.stream.Log(akka://aws-akka-http/system/Materializers/StreamSupervisor-3)] [client-plain-text FromNet] Element: SessionBytes  ByteString(208 bytes) first + last 100:
 48 54 54 50 2F 31 2E 31 20 34 32 36 20 55 70 67  | HTTP/1.1 426 Upg
 72 61 64 65 20 52 65 71 75 69 72 65 64 0D 0A 43  | rade Required..C
 6F 6E 74 65 6E 74 2D 54 79 70 65 3A 20 74 65 78  | ontent-Type: tex
 74 2F 68 74 6D 6C 3B 63 68 61 72 73 65 74 3D 69  | t/html;charset=i
 73 6F 2D 38 38 35 39 2D 31 0D 0A 43 6F 6E 74 65  | so-8859-1..Conte
 6E 74 2D 4C 65 6E 67 74 68 3A 20 35 39 0D 0A 43  | nt-Length: 59..C
 6F 6E 6E 65                                      | onne
                     ... [8 bytes omitted] ...
 6C 6F 73 65 0D 0A 53 65 72 76 65 72 3A 20 4A 65  | lose..Server: Je
 74 74 79 28 39 2E 34 2E 35 31 2E 76 32 30 32 33  | tty(9.4.51.v2023
 30 32 31 37 29 0D 0A 0D 0A 3C 68 31 3E 42 61 64  | 0217)....
Bad
 20 4D 65 73 73 61 67 65 20 34 32 36 3C 2F 68 31  |  Message 426

reason: Up
 67 72 61 64 65 20 52 65 71 75 69 72 65 64 3C 2F  | grade Required</
 70 72 65 3E                                      | pre>

So I agree that we need to use the "Connection-Level Client-Side API", but we need to find a way so that the connection is not recreated every time...

Honestly, I think we can close this, and revisit this when akka-http provides http/2 support in the "Request-Level Client-Side API"

@jtjeferreira @matsluni I logged https://github.com/apache/incubator-pekko-http/issues/483 - is this the missing feature that you need?

@jtjeferreira @matsluni I logged apache/incubator-pekko-http#483 - is this the missing feature that you need?

My last comment referred to another limitation:

Honestly, I think we can close this, and revisit this when akka-http provides http/2 support in the "Request-Level Client-Side API"

i.e currently http/2 is only supported at the Connection-Level API:
val connectionFlow: Flow[HttpRequest, HttpResponse, Future[Http.OutgoingConnection]] =
      Http().connectionTo("localhost").http2()

but it should support the Request-Level API:
val httpResponseFuture: Future[HttpResponse] = Http().singleRequest(HttpRequest(uri = "http://localhost"))
//hipotethical API
val http2ResponseFuture: Future[HttpResponse] = Http2().singleRequest(HttpRequest(uri = "http://localhost"))


paths:contentPath is oddly set when Ghost is installed via NPM

Issue Summary
When Ghost is installed via NPM, the paths:contentPath is set to (project folder)/node_modules/ghost/content, not (project folder)/content. But database:connection:filename is set correctly.
It seems makePathsAbsolute is written in wrong way.
I worked around this issue through setting paths:contentPath to content not content/, but it is not a good solution of course.
Steps to Reproduce

Install Ghost via NPM
Set paths:contentPath to content/

Technical details:

Ghost Version: 1.2.0
Node Version: 6.11.1
Browser/OS: Manjaro
Database: SQLite3

I think this is a duplicate of https://github.com/TryGhost/Ghost/issues/8754, although this provides some nice extra info - thanks.
Would love a PR to fix this :)


504 Gateway timeout errors

Hi - I'm getting 504 timeout errors and general performance issues, especially when trying to access wp-admin across various sites within auto DNS. Any ideas? I'm on:
OSX High Sierra
Docker Version 18.03.0-ce-mac60 (23751)
Compose: 1.20.1
Here's my .env attached
env.txt

Seeing the same thing with Docker Engine: 18.03.1-ce, Compose: 1.21.1. Recently updated docker and an apple security patch which required a restart. Now getting 504 gateway timeout errors on /wp-admin.

hmm, well, went to dive in to the issue this evening and it's working now ¯_(ツ)_/¯ it may have been the network I was on today. I will check again tomorrow.

@stefanlesik please attach output from docker-compose logs as well as the log files from ./log/ for your httpd and php version.

Same on Windows 10, Docker 18.03.1-ce-win65 (17513).

Same on Manjaro, 504, restart of docker (sql) temp fix it

@JakubHelis  @adammarton @cnewfeldt @stefanlesik
I just merged an update for the webserver. Could you try again please after pulling latest master branch.

@cytopia
For me it's still the same. + reloading the query (refresh page, retry ajax)  mostly gives positive result ( working)

@cytopia  Interesting fact - The troubles gone when i switch from apache 2.4 to ngnix.   ( newest version of devilbox)

@JakubHelis thanks for the heads up

I also have mac users using my setup where devilbox just gets "angry" and my php code stops responding....usually fixed by a docker-compose stop && docker-compose start

Host machine with Linux Manjaro, also experiencing 504 Gateway on admin panel of Prestashop projects. Seems better after changing Apache 2.4 to 2.2.

Anyone having this issue on a different web server than Apache 2.4?

Yes, on nginx-stable, but this might be because of my raped WordPress instance.

What about increasing timeout value of Apache 2.4
Create this file: cfg/apache-2.4/timeout.cfg
Timeout 600

in my issue, the code running should not take 60 seconds, but when this occurs, its like php fpm is frozen. And changing the timeout would just make it wait longer before reporting back.
I wonder if it may be related to database connections used in the php code when it hangs.

I might have found a proxy timeout overwrite by adding a vhost-gen template to the project which has timeouts (How to here: https://devilbox.readthedocs.io/en/latest/configuration-project/custom-vhost.html)
If this is the catch I will increase the Proxy timeout by default. Please let me know how it goes
The specific line is this:
    # enablereuse requires Apache 2.4.11 or later
    <Proxy "fcgi://php:9000/" enablereuse=on max=10>
        ProxySet timeout=3600
        ProxySet connectiontimeout=3600
    </Proxy>

The full template is for apache24.yml is here:
---

# Apache 2.4 vHost Template defintion for vhost-gen.py
#
# The 'feature' section contains optional features that can be enabled via
# conf.yml and will then be replaced into the main vhost ('structure' section)
# into their corresponding position:
#
#    __XDOMAIN_REQ__
#    __PHP_FPM__
#    __ALIASES__
#    __DENIES__
#    __STATUS__
#
# The features itself also contain variables to be adjusted in conf.yml
# and will then be replaced in their corresponding feature section
# before being replaced into the vhost section (if enabled):
#
# PHP-FPM:
#    __PHP_ADDR__
#    __PHP_PORT__
# XDomain:
#    __REGEX__
# Alias:
#    __REGEX__
#    __PATH__
# Deny:
#    __REGEX__
# Status:
#    __REGEX__
#
# Variables to be replaced directly in the vhost configuration can also be set
# in conf.yml and include:
#    __VHOST_NAME__
#    __DOCUMENT_ROOT__
#    __INDEX__
#    __ACCESS_LOG__
#    __ERROR_LOG__
#    __PHP_ADDR__
#    __PHP_PORT__
#

###
### Basic vHost skeleton
###
vhost: |
  <VirtualHost __DEFAULT_VHOST__:__PORT__>
      ServerName   __VHOST_NAME__

      CustomLog  "__ACCESS_LOG__" combined
      ErrorLog   "__ERROR_LOG__"

  __REDIRECT__
  __SSL__
  __VHOST_DOCROOT__
  __VHOST_RPROXY__
  __PHP_FPM__
  __ALIASES__
  __DENIES__
  __SERVER_STATUS__
      # Custom directives
  __CUSTOM__
  </VirtualHost>

###
### vHost Type (normal or reverse proxy)
###
vhost_type:
  # Normal vHost (-p)
  docroot: |
    # Define the vhost to serve files
    DocumentRoot "__DOCUMENT_ROOT__"
    <Directory "__DOCUMENT_ROOT__">
        DirectoryIndex __INDEX__

        AllowOverride All
        Options All

        RewriteEngine on
        RewriteBase /

        Order allow,deny
        Allow from all
        Require all granted
    </Directory>

  # Reverse Proxy (-r)
  rproxy: |
    # Define the vhost to reverse proxy
    ProxyRequests off
    ProxyPass __LOCATION__ __PROXY_PROTO://__PROXY_ADDR__:__PROXY_PORT__
    ProxyHTMLURLMap __PROXY_PROTO://__PROXY_ADDR__:__PROXY_PORT__ __LOCATION__
    <location __LOCATION__>
        ProxyPassReverse /
        SetOutputFilter  proxy-html
        ProxyHTMLURLMap  /      __LOCATION__
        ProxyHTMLURLMap  __LOCATION__  __LOCATION__
        RequestHeader    unset  Accept-Encoding
    </location>

###
### Optional features to be enabled in vHost
###
features:

  # SSL Configuration
  ssl: |
    SSLEngine on
    SSLCertificateFile    "__SSL_PATH_CRT__"
    SSLCertificateKeyFile "__SSL_PATH_KEY__"
    SSLProtocol           __SSL_PROTOCOLS__
    SSLHonorCipherOrder   __SSL_HONOR_CIPHER_ORDER__
    SSLCipherSuite        __SSL_CIPHERS__

  # Redirect to SSL directive
  redirect: |
    RedirectMatch (.*) https://__VHOST_NAME__:__SSL_PORT__$1

  # PHP-FPM will not be applied to a reverse proxy!
  php_fpm: |
    # PHP-FPM Definition
    <FilesMatch \.php$>
        Require all granted
        SetHandler proxy:fcgi://__PHP_ADDR__:__PHP_PORT__
    </FilesMatch>

    # enablereuse requires Apache 2.4.11 or later
    <Proxy "fcgi://__PHP_ADDR__:__PHP_PORT__/" enablereuse=on max=10>
        ProxySet timeout=3600
        ProxySet connectiontimeout=3600
    </Proxy>

    # If the php file doesn't exist, disable the proxy handler.
    # This will allow .htaccess rewrite rules to work and
    # the client will see the default 404 page of Apache
    RewriteCond %{REQUEST_FILENAME} \.php$
    RewriteCond %{DOCUMENT_ROOT}/%{REQUEST_URI} !-f
    RewriteRule (.*) - [H=text/html]

  alias: |
    # Alias Definition
    Alias "__ALIAS__" "__PATH____ALIAS__"
    <Location "__ALIAS__">
    __XDOMAIN_REQ__
    </Location>
    <Directory "__PATH____ALIAS__">
        Order allow,deny
        Allow from all
        Require all granted
    </Directory>

  deny: |
    # Deny Definition
    <FilesMatch "__REGEX__">
        Order allow,deny
        Deny from all
    </FilesMatch>

  server_status: |
    # Status Page
    <Location __REGEX__>
        SetHandler server-status
        Order allow,deny
        Allow from all
        Require all granted
    </Location>

  xdomain_request: |
    # Allow cross domain request from these hosts
    SetEnvIf Origin "__REGEX__" AccessControlAllowOrigin=$0
    Header add Access-Control-Allow-Origin %{AccessControlAllowOrigin}e env=AccessControlAllowOrigin
    Header always set Access-Control-Allow-Methods "POST, GET, OPTIONS, DELETE, PUT"
    Header always set Access-Control-Max-Age "0"
    Header always set Access-Control-Allow-Headers "x-requested-with, Content-Type, origin, authorization, accept, client-security-token"
    # Added a rewrite to respond with a 200 SUCCESS on every OPTIONS request.
    RewriteEngine On
    RewriteCond %{REQUEST_METHOD} OPTIONS
    RewriteRule ^(.*)$ $1 [R=200,L]

Once the template is applied, restart the devilbox and enter the webserver (docker-compose exec httpd bash) to see if those changes are actually picked up for the speicifc vhost

Also let me now after what time the Gateway Timeout error appears (roughly in seconds).
What did you set your PHP's max_execution_time to?

@qlstudent

did you try this: https://github.com/cytopia/devilbox/issues/234#issuecomment-399342207?
Please also provide more information so that I can actually reproduce this issue:

I have the same issue as well in Mac OS El Capitan. I switched from PHP 7.2 to PHP 5.6 and I have 504 error

For intranet or projects? If for projects, please let me know which Framework and for what requests

and also some sites show up as they don't have /etc/hosts entry even though they totally do.

Can you attach a screenshot of the vhost overview with open console

@cytopia I was able to fix / mitigate my issue by changing some php fpm settings.
I've read some things that say php-fpm children have problems after handling enough requests and its better if it is set to stop and start a replacement child.
I used ab (apache benchmark) to cause the errors, and after these pool changes, I cannot reproduce it.
ab -r -n 5000 -c 100 https://devilbox.loc/vhosts.php
(Note: this benchmarking will use all your system resources. my CPU fans were going at full speed while running this)
pool.conf changes:
pm.max_children = 100
pm.start_servers = 20
pm.max_requests = 500

I get these as well with wordpress.

student@student-HP-EliteDesk-800-G1-SFF:~$ cat src/devilbox/.env
###
###  ---------------------------------------------------
###  D E V I L B O X   R U N - T I M E   S E T T I N G S
###  ---------------------------------------------------
###
###  All the following settings are applied during
###  $ docker-compose up
###
###  No need to rebuild any dockers!
###
###  For custom variables, scroll to the bottom
###

###
### Show all executed commands in each
### docker image during docker-compose up?
###
### 1: Yes
### 0: No
DEBUG_COMPOSE_ENTRYPOINT=1

###
### Log to file or Docker logs.
###
### Logging to file means log files are available under log/
### on your host operating system.
### Logging to Docker logs means log files are streamed to
### stdout and stderr.
###
### 1: Log to Docker logs
### 0: Log to file
###
DOCKER_LOGS=0

###
### Relative or absolute path to the devilbox repository.
### (Used as a prefix for all mount paths)
### There is no need to change this.
###
### The only exception is for OSX users wanting to use NFS
### mounts instead of Filesystem mounts due to degraded performance
### on OSX.
###
### Note: When changing this variable you must re-create the container.
###       Simply remove it, it will be auto-created during the next start:
###
### $ docker-compose rm -f
###
DEVILBOX_PATH=.

###
### At what IP address should the docker services listen
### on the Host computer?
###
### The specified default should be fine for Linux and OSX (<IP_ADDRESS>:).
### If you are on windows, you will probably have to change
### it to the IP address of the docker machine.
###
### a.) Leave blank, to listen on all interfaces (no trailing color ':')
###    LOCAL_LISTEN_ADDR=
### b.) If an IP is specified, note the trailing colon ':'
###    LOCAL_LISTEN_ADDR=<IP_ADDRESS>:
###
LOCAL_LISTEN_ADDR=

###
### This is the domain suffix your projects will be made available
### with mass-virtual-hosting.
### It is also required for the internal DNS server to be setup.
###
### Note: Only ALPHA ([a-zA-Z]+) characters are supported.
###
### Example:
###   TLD_SUFFIX=loc
### Makes your project available under xxxx.loc
###
### Example:
###   TLD_SUFFIX=local
### Makes your project available under xxxx.local
###
TLD_SUFFIX=loc

###
### Optional DNS configuration
### Allows you to add extra DNS records (above the wildcard entry)
### Useful if your host computer run other Docker services that you want to connect to or reach
### from within the Devilbox network by a custom hostname.
###
### Format:
### -------
### Resolve any custom defined hostname to an IP address (useable inside container and host os)
###     EXTRA_HOSTS=<hostname>=<ip>[,<hostname>=<ip>]
###
### Resolve any custom defined hostname to whatever IP address a CNAME resolves to
### (Useable inside the container and host OS).
### Note: CNAME must be resolvable by Google DNS
###     EXTRA_HOSTS=<hostname>=<CNAME>[,<hostname>=<CNAME>]
###
### Examples:
### ---------
### EXTRA_HOSTS=hostname.loc=<IP_ADDRESS>
### EXTRA_HOSTS=host.loc=<IP_ADDRESS>,host.example.org=<IP_ADDRESS>
EXTRA_HOSTS=

###
### Set your user id and group id
###
### This should be changed to the value of your local
### users uid and gid
###
### Type `id` on the terminal to find out your values
###
NEW_UID=1000
NEW_GID=1000

###
### Timezone for all dockers and service config files
###
TIMEZONE=Europe/Berlin

################################################################################
###
### INTRANET SETTINGS
###
################################################################################

###
### TLD_SUFFIX domains are checked if they are set in the
### host computer /etc/hosts or available via attached DNS server.
### Timeout is done on vhosts.php (intranet) via ajax calls.
### In order to keep performance, set this to a low value.
### DNS checks might not succeed in time on slow machines.
### If DNS is valid, but timeout is expired, set this to a higher value.
###
### DNS_CHECK_TIMEOUT value is how many seconds to time out
### Default is to timeout after 1 second (DNS_CHECK_TIMEOUT=1)
###
DNS_CHECK_TIMEOUT=1

###
### Devilbox UI SSL Certificate generation
###
### When using SSL each certificate requires names for which it is responsible:
### Common Name as well as alternative names.
###
### Specify comma separated hostnames below by which you want to access the Devilbox.
### Those hostnames will be included in the SSL certificate for the Devilbox intranet.
### This has nothing to do for SSL certificates for projects, it is just for the intranet
### itself.
###
DEVILBOX_UI_SSL_CN=localhost,*.localhost,devilbox,*.devilbox

###
### Devilbox UI Password protection enable/disable (1/0)
###
### Set DEVILBOX_UI_PROTECT to 1 in order to password protect the
### intranet.
###
### Example:
###   DEVILBOX_UI_PROTECT=1
###   DEVILBOX_UI_PROTECT=0
###
DEVILBOX_UI_PROTECT=0

###
### Devilbox UI Password
###
### When DEVILBOX_UI_PROTECT=1, use the following password
### to log in. The password can always be changed.
### When changing the password, make sure to restart your
### PHP container.
###
### Example:
###   DEVILBOX_UI_PASSWORD=my-very-secure-password
###   DEVILBOX_UI_PASSWORD=Some pass with spaces
###
### The default username is 'devilbox'
###
DEVILBOX_UI_PASSWORD=password

###
### Enable the Devilbox Intranet?
###
### Example:
###   DEVILBOX_UI_ENABLE=1
###   DEVILBOX_UI_ENABLE=0
###
DEVILBOX_UI_ENABLE=1

################################################################################
###
### 1. Choose Images (Version)
###
################################################################################

###
### You can choose any combination of httpd, mysql, postgresql or php.
 Host Mounts (Your computer)
###
################################################################################

###
### Local filesystem path to www projects.
###
### Note: When changing this variable you must re-create the container.
###       Simply remove it, it will be auto-created during the next start:
###
### $ docker-compose rm -f
###
HOST_PATH_HTTPD_DATADIR=./data/www

###
### Local filesystem path to mysql/mariadb datadir.
###
### This can be an existing mysql data directory or empty.
### If it already is a mysql data directory with content,
### it will be mounted into the docker and used.
###
### If this directory is empty, a new mysql database will be
### created.
###
### Note: Inside this path, a subdirectory with the mysql|mariadb
###       version will be created where the actual data resides.
###       This is to protect databases from being altered by
###       newer or older mysql|mariadb server versions.
###
### Note: When changing this variable you must re-create the container.
###       Simply remove it, it will be auto-created during the next start:
###
### $ docker-compose rm -f
###
HOST_PATH_MYSQL_DATADIR=./data/mysql

###
### Local filesystem path to postgresql datadir.
###
### This can be an existing postgresql data directory or empty.
### If it already is a postgresql data directory with content,
### it will be mounted into the docker and used.
###
### If this directory is empty, a new postgresql database will be
### created.
###
### Note: Inside this path, a subdirectory with the postgresql
###       version will be created where the actual data resides.
###       This is to protect databases from being altered by
###       newer or older postgres server versions.
###
### Note: When changing this variable you must re-create the container.
###       Simply remove it, it will be auto-created during the next start:
###
### $ docker-compose rm -f
###
HOST_PATH_PGSQL_DATADIR=./data/pgsql

###
### Local filesystem path to mongodb datadir.
###
### This can be an existing mongodb data directory or empty.
### If it already is a mongodb data directory with content,
### it will be mounted into the docker and used.
###
### If this directory is empty, a new mongodb database will be
### created.
###
### Note: When changing this variable you must re-create the container.
###       Simply remove it, it will be auto-created during the next start:
###
### $ docker-compose rm -f
###
HOST_PATH_MONGO_DATADIR=./data/mongo

################################################################################
###
### 3. PHP Docker Settings
###
################################################################################

###
### Enable certain PHP modules which are not enabled by default
###
### Currently the only module that can be enabled is 'ioncube'
###
### PHP_MODULES_ENABLE=ioncube
###
PHP_MODULES_ENABLE=

###
### Disable any PHP modules that you don't require
###
### Specify a comma separated list without spaces of modules to disable
###
### PHP_MODULES_DISABLE=xdebug,imagick,swoole
###
PHP_MODULES_DISABLE=

###
### Configure everything else about PHP in
### * cfg/php-ini-X.X/*.ini
### * cfg/php-fpm-X.X/*.conf

################################################################################
###
### 4. HTTPD Docker Settings
###
################################################################################

###
### Expose HTTPD Port to Host
###
HOST_PORT_HTTPD=80
HOST_PORT_HTTPD_SSL=443

###
### Document Root Subdirectory
###
### In your project directory, which subfolder should
### serve your files?
###
### When changing this value, restart the devilbox.
###
HTTPD_DOCROOT_DIR=htdocs

###
### Per vHost Config Subdirectory
###
### In your project directory, which subfolder should
### hold apache, nginx templates for a customized vhost?
###
### When changing this value, restart the devilbox.
###
HTTPD_TEMPLATE_DIR=.devilbox

################################################################################
###
### 5. MySQL Docker Settings
###
################################################################################

###
### MySQL root user password
###
### If $HOST_PATH_MYSQL_DATADIR already contains an existing
### mysql datadir, enter the password for the existing mysql database
###
### If $HOST_PATH_MYSQL_DATADIR is empty, choose a new password that
### will be applied
###
MYSQL_ROOT_PASSWORD=

###
### Custom MySQL Runtime Settings
###
MYSQL_GENERAL_LOG=0

###
### Expose MySQL Port to Host
###
HOST_PORT_MYSQL=3306

################################################################################
###
### 6. PostgreSQL Docker Settings
###
################################################################################

###
### PostgreSQL 'root' user name (usually postgres)
###
PGSQL_ROOT_USER=postgres

###
### PostgreSQL 'root' user password
###
PGSQL_ROOT_PASSWORD=

###
### Expose PostgreSQL Port to Host
###
HOST_PORT_PGSQL=5432

################################################################################
###
### 7. Redis Docker Settings
###
################################################################################

###
### Expose Redis Port to Host
###
HOST_PORT_REDIS=6379

################################################################################
###
### 8. Memcached Docker Settings
###
################################################################################

###
### Expose Memcached Port to Host
###
HOST_PORT_MEMCD=11211

################################################################################
###
### 9. MongoDB Docker Settings
###
################################################################################

###
### Expose MongoDB Port to Host
###
HOST_PORT_MONGO=27017

################################################################################
###
### 10. Bind Docker Settings
###
################################################################################

###
### Expose Bind Port to Host
###
HOST_PORT_BIND=1053

###
### Add comma separated DNS server from which you want to receive DNS
### You can also add DNS servers from your LAN (if any are available)
###
BIND_DNS_RESOLVER=<IP_ADDRESS>,<IP_ADDRESS>

###
### Validate DNSSEC
###
### Values:
###  no:    DNSSEC validation is disabled
###  yes:   DNSSEC validation is enabled, but a trust anchor must be manually configured.
###  auto:  DNSSEC validation is enabled, and a default trust anchor for root zone is used.
###
BIND_DNSSEC_VALIDATE=no

###
### Bind timing options (time in seconds)
###
### Leave empty for defaults.
### Only change when you know what you are doing.
###
BIND_TTL_TIME=
BIND_REFRESH_TIME=
BIND_RETRY_TIME=
BIND_EXPIRY_TIME=
BIND_MAX_CACHE_TIME=

###
### Show DNS Queries in Docker logs output?
###
### 1: Yes
### 0: No
BIND_LOG_DNS_QUERIES=0

################################################################################
###
### 11. Custom variables
###
################################################################################

###
### Any variable defined in this file will be available
### as environment variables to your PHP/HHV Docker container.
###
### This might be useful to set application environment and retrieve
### them via: <?php getenv('MY_APPLICATION_ENV'); ?>
###

###
### Example:
### <?php echo getenv('Foo'); ?> would produce: 'some value'
###
#Foo=some value
student@student-HP-EliteDesk-800-G1-SFF:~$

Successfully tested settings
I have now benchmarked it with PrestaShop and Magento as they are pretty performance hungry.
Additionally only tested HTTPS to get even more performance penalty:

https://devilbox.readthedocs.io/en/latest/examples/setup-presta-shop.html
https://devilbox.readthedocs.io/en/latest/examples/setup-magento.html

Container:

PHP: PHP-FPM 7.1
HTTPD: Apache 2.4

HTTPS with keep alive
ab -r -n 5000 -c 100 -v 2 -k  https://presta.loc/en/ 2>/dev/null | grep ^HTTP
ab -r -n 5000 -c 100 -v 2 -k  https://magento/ 2>/dev/null | grep ^HTTP

HTTPS without keep alive
ab -r -n 5000 -c 100 -v 2 https://presta.loc/en/ 2>/dev/null | grep ^HTTP
ab -r -n 5000 -c 100 -v 2 https://magento/ 2>/dev/null | grep ^HTTP

After a lot of back and forth I now came to the following balance between initial RAM consumption and not throwing 500s:
[www]
pm.max_children = 50
pm.start_servers = 10
pm.min_spare_servers = 5
pm.max_spare_servers = 15
pm.max_requests = 500

Please test
@stefanlesik @cnewfeldt @adammarton @JakubHelis @science695 @pauloffb @qlstudent
Can you please test the new PHP-FPM configuration and report back how this works.
How to apply the configuration?
To set this up read on here: https://devilbox.readthedocs.io/en/latest/configuration-files/php-fpm-conf.html

As far as I can tell (after a day of work) the config fixes the problem.

@chrizz1001 good to hear.
I will have another few commits coming soon to generalize the configuration accross all PHP versions. This will lead to new Docker images. I will update here once they are pushed to the above specified PR.

Another thing I just found out is that Apache 2.4 is keeping the connection to the PHP-FPM processes up, thus the PHP-FPM processes cannot respawn and get not drowned. I will add a patch for it as well.

Changed default settings for performance
I have now switched from pm = dynamic to pm = ondemand. This has the benefit that the initial RAM consumption of the PHP container is at around 50-60 Megabytes in total.
Workers are spawned ondemand and will also be released after 10sec of inactivity, meaning if nothing happens the RAM consumption goes back to 50-60 Megabytes.
Nevertheless I have benched it with ab and the results are very promising.
Defaults for PHP are available here:

https://github.com/devilbox/docker-php-fpm/blob/master/build/ansible/group_vars/all.yml#L26
https://github.com/devilbox/docker-php-fpm/tree/master/build/ansible/CONFIGURATION

php-fpm.conf
[global]
log_level = notice

[www]
request_terminate_timeout = 120s
listen.backlog = 128
pm = ondemand
pm.max_children = 50
pm.start_servers = 4
pm.min_spare_servers = 2
pm.max_spare_servers = 6
pm.max_requests = 500
pm.process_idle_timeout = 10s

php.ini
memory_limit		= 512M

max_execution_time      = 120
max_input_time          = 120

post_max_size           = 72M
upload_max_filesize     = 64M
max_file_uploads        = 20

variables_order         = EGPCS
max_input_vars          = 8000
max_input_nesting_level = 64

error_reporting         = E_ALL | E_NOTICE | E_STRICT | E_DEPRECATED
xmlrpc_errors           = Off
report_memleaks         = On
display_errors          = On
display_startup_errors  = On
track_errors            = On
log_errors              = On
html_errors             = On

xdebug.default_enable   = Off
xdebug.profiler_enable  = Off
xdebug.remote_enable    = Off
xdebug.remote_autostart = Off

Web server changes
Those will be available later this day in a new PR.

Travis-CI seems to have some problems and therefore the images are not yet available: https://www.traviscistatus.com/incidents/118bjpkgkg00

liste.backlog = 128 was way too low. I've set it to 1024 now and waiting for the images to be pushed to Dockerhub.

PR is up: https://github.com/cytopia/devilbox/pull/338


Add restricted mode - only installer or whitelisted admins can mint /…

... Wrong place...

My bad.. wrong folk


[Bug] Precondition aggregate is empty

Hi,
So I was adding a precondition to prevent changing a field of an aggregate to the same value
cqrs
  .definePreCondition({
    name: ['renameNode'],
    description: 'Cannot rename to same.',
    priority: 1
  }, (cmd, agg, cb) => {
    if (agg.get('label') === cmd.payload.label) {
      return cb(new Error())
    }
    cb(null);
  })

and guess what? The pre-condition aggregate doesn't have the attributes loaded.
The command works fine and if I log something in the callback, it shows.
I've debugged for 3 hours already and I think the issue is in defaultCommandHandler.js somewhere within loadAggregate: function.
What am I missing?

My aggregate looks like this.
cqrs.defineAggregate(
    {
      name: 'node',
      defaultEventPayload: 'payload'
    },
    {} // default starting data for the aggregate
  )

and my domain thingy is
require('cqrs-domain')({
...
})
  .defineCommand({
    aggregateId: 'aggregate.id',
    aggregate: 'aggregate.name',
    payload: 'payload',
    context: 'context.name',
  })
  .defineEvent({
    name: 'event',
    aggregateId: 'aggregate.id',
    aggregate: 'aggregate.name',
    context: 'context.name',
  });

Never mind, I'm stupid and haven't seen that the event is messed up:
require('cqrs-domain').defineEvent(
  {
    version: 0,
    meta: 'meta'
  },
  (cmd, aggregate) => aggregate.set(cmd.payload)
);

and it should be (note the cb function):
require('cqrs-domain').defineEvent(
  {
    version: 0,
    meta: 'meta'
  },
  (cmd, aggregate) => aggregate.set(cmd.payload)
);

I hope I didn't waste anyone's time. Thanks! :)


Adding rp_IFrame and polling client

Changes are added via https://github.com/wso2/product-is/pull/437..  hence closing.


chore: adds deprecation alerts

adds deprecation alerts for .form-control-rg and .form-control-md as well as .form-select-rg and .form-select-md to documentation and demo

@b1aserlu I have chaned the sonarcloud settings. Now you should not get a duplication warning for textarea.stories.ts and select.stories.ts anymore.
If the SonarCloud Code Analysis check is not rerunning until next week, try to push a small change (which you can revert afterwards) to execute all checks again.


how to train model with custom data?

Thank you for your job! I want to know how to train cps with my custom dataset, i should change code where it is?

Hi, just change the Dataset Class such as here, and change something related to your dataset in the config file.


some reformattting for flake8 etc

@casperdcl could you help with the following 3 types of problems thrown up by the yaml checker?
could not determine a constructor for the tag '!record'
  in "model/DetectorInformation.yml", line 2, column 21

expected a single document in the stream
  in ".clang-format", line 2, column 1
but found another document
  in ".clang-format", line 3, column 1

python/petsird_generator.py:105:5: E125 continuation line with same indent as next logical line
p

The latter were reformatted by black presumably.

The "could not determine a constructor for the tag" issue is mentioned in various places, including https://github.com/yaml/pyyaml/issues/825. They all seem to say we'd need a Python class for the checker to be happy, which seems a no-go. https://github.com/pre-commit/pre-commit-hooks?tab=readme-ov-file#check-yaml doesn't give a lot of options, so I'll try --unsafe. That might prevent the clang-format error as well.


CREATE DATABASE ON

The following page describes the concept of storing database files in Azure using blob storage: https://docs.microsoft.com/en-gb/sql/relational-databases/databases/sql-server-data-files-in-microsoft-azure
However, the syntax of CREATE DATABASE does not apparently allow for specifying a URL.
Can this page be updated with the valid syntax and an example, and also a reference to the concept page mentioned above?

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9e8abfde-2523-5c70-bb95-b10101a726ff
Version Independent ID: 3b8152d2-b49e-df4b-0f05-3ec52aa5bb71
Content: CREATE DATABASE (Transact-SQL) - SQL Server
Content Source: docs/t-sql/statements/create-database-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @CarlRabeler
Microsoft Alias: carlrab

@banzaisi  Thank you for the feedback.
Assigning this to the author for review.

@banzaisi Thank you for your question. @CarlRabeler is currently out of the office. @MashaMSFT can you please take a look at this suggestion.
@banzaisi Can I ask a few questions? So you're saying that the article shows using CREATE DATABASE with a URL to blob storage, but when you do this it does not work? Do you have an error you could share? So you're asking us to verify that the article works and/or change the article to show the correct syntax for creating a database in Azure blob storage. Is this correct? Thank you!

Hi @banzaisi , thank you for contacting Microsoft! Sorry, would you mind clarifying your question? The doc  you mention has this code sample:
CREATE CREDENTIAL [https://testdb.blob.core.windows.net/data]
WITH IDENTITY='SHARED ACCESS SIGNATURE',
SECRET = '<your SAS key>'

CREATE DATABASE testdb
ON
( NAME = testdb_dat,
    FILENAME = 'https://testdb.blob.core.windows.net/data/TestData.mdf' )
 LOG ON
( NAME = testdb_log,
    FILENAME =  'https://testdb.blob.core.windows.net/data/TestLog.ldf')

Is that different than what you were looking for?
Thanks!
Masha from the SQL Docs team

@banzaisi , oh, I just reread your question :) Are you asking to have the above example added to the CREATE DATABASE article?
Thanks!
Masha from the SQL Docs team

@MashaMSFT , yes, exactly. Does the main syntax for CREATE DATABASE also need clarifying to say that Azure blob URLs are also allowed, per below?
Are all the other options like SIZE, MAXSIZE and FILEGROWTH also supported for Azure blobs?
 ::=
{
(
NAME = logical_file_name ,
FILENAME = { 'os_file_name' | 'filestream_path' | 'azure_blob_url' }
[ , SIZE = size [ KB | MB | GB | TB ] ]
[ , MAXSIZE = { max_size [ KB | MB | GB | TB ] | UNLIMITED } ]
[ , FILEGROWTH = growth_increment [ KB | MB | GB | TB | % ] ]
)
}

Hi @banzaisi , thanks for the confirmation! Let me verify that the other options are supported, and then I'll update the doc :) Thanks again for helping improve our documentation!
Masha from the SQL Docs team

Hi @banzaisi , thanks again for providing feedback to our documentation! I have created an internal work item to track this work (1622218). I will close out this Git Issue for the time being, but please feel free to respond should you need anything further in the meantime.
Thanks!
Masha from the SQL Docs team


feat(platform/website): profile pictures

Proposed changes
This pr implements website support for profile pictures.
Including:

Displaying and subscribing to changes
Uploading profile pictures
Removing profile pictures

Types of changes
What types of changes does your code introduce to Scuffle?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

Further comments

The slash in parenthesis of the commit message is for sure not to format. Please choose one or remove the parenthesis entirely if the scope is global.

The slash in parenthesis of the commit message is for sure not to format. Please choose one or remove the parenthesis entirely if the scope is global.

Because this is a mono repo the scope is not global most of the time. What scope would you suggest for platform/website or platform/api?

@lennartkloock Hmm, I've googled around a bit and it looks like dashes are generally uhhh ok, probably.
Conventional Commits is strongly against anything but a single word:

A scope MAY be provided after a type. A scope MUST consist of a noun describing a section of the codebase surrounded by parenthesis, e.g., fix(parser):

Karma Style is indifferent:

Example <scope> values:

web-server

I personally like the slash, it clearly conveys the intent - "part of" or a "subdirectory", instead of just "component name". A dash is a compromise I'm willing to accept (e.g. platform-api).
On a related note, I think the prefix of the commit message is getting too long with that subject name ij general. Maybe we should name the components somehow and use that shorter name? How about shortening website to web? Not much help though... Still, shorter is better in this case IMO.

I just noticed the image_processor commits. Not sure how I feel about that underscore... Probably negatively. If those will be squashed anyway then it doesn't matter much.

@IS2511 I'll just leave out platform/ or video/ in the future. I guess that's enough because apart from api all components have unique names anyway


Migrate scanner to C

Hey, we are thinking about bundling this parser into Neovim, however one of the concerns is raising our build requirements to include C++. (https://github.com/neovim/neovim/pull/22458)
I've looked at the code and it would be quite straightforward to port to C. We've already done this for https://github.com/MDeiml/tree-sitter-markdown/issues/84 which has a larger scanner.
Is this something that would be accepted?
Duplicate issue on tree-sitter-bash: https://github.com/tree-sitter/tree-sitter-bash/issues/149

I'd suggest to take Haskell's scanner as a reference.


ANSI escape sequences are not rendered correctly in Windows Console, Powershell and Powershell Core

Operating system
Windows 10 21H2
Kluctl Version
2.12.7
Kubernetes Version
1.19.7
kubectl Version
1.25.0
Bug description
The Windows Console (cmd.exe) and also Windows Powershell don't understand ANSI escape squences from KluCTL, therefore their codes are shown in the outputs of KluCTL.
Windows Console (cmd.exe):

Windows Powershell:

Windows Powershell Core (tested in version 7.2):

Hint: Even if Powershell Core has the VT support enabled for itself by default, it seems it is not recognizing the outputs of programs which are being started from it. Piping them via Out-Host renders them correctly, see https://stackoverflow.com/a/51681675 for details.
Steps to reproduce

Start a Powershell /  cmd session
Do a KluCTL diff against a cluster
Observe the outputs

Relevant log output
No response

This was fixed in v2.15.0. Colors should be properly supported in Windows now.


VFS and LWIP should share file/socket descriptor space

Currently some functions of the POSIX file API are routed to VFS and those that are partof the BSD socket API are routed to LWIP. this is very confusing. for example, it is possible to open a file, but subsequent read will fail because it goes to LWIP.
of course, one can side-step the issue by using buffered file i/o (fopen, fread), but it's not ideal. it should be possible to route LWIP through VFS and make them share descriptor space.

Hi, @rojer. I think you can consider the LWIP build option LWIP_SOCKET_OFFSET to let socket descriptor starting from an offset to avoid the descriptor overlap with VFS file descriptor.

the issue is not overlap of descriptor space, but the fact that read is routed to LWIP only and cannot be used for operations on VFS files.

Okay.

I also have run in to this problem.  It's very serious because it means basically that libc is linking unreliably.  It's not at all clear which read or write will be linked to a component.  LWIP_SOCKET_OFFSET should make it easy enough to fix inside of newlib, but that call routing has to be done at runtime.

@igrr, in a forum post from half a year ago, you mentioned that you'll fix the LWIP macro issues with this ticket. As far as I see in the closing commit, that was not the case.
I know, there is the #undef hack, but frankly it's very annoying to have issues with such common indentifiers. Not only bind, but also the lack of send is quite annoying.
Any chance to fix this, perhaps attach it to another ticket?

@chanibal

you mentioned that you'll fix the LWIP macro issues with this ticket. As far as I see in the closing commit, that was not the case.

Sorry we missed this comment on this closed issue until just now. I agree we should fix this properly. Could you please open a new issue to keep track of this one?


lokalkauf Logo is cut off in Merklisten view

The fallback preview (lokalkauf logo) is cut off in the list view (see screenshot)

I used the social media image that has not the right form factor.
We do not have a design for that case yet, so waiting for an image that is to be displayed here :)

bigger logo, but still need a new design for that

Tested in Android, ios and its okay


Licence

Just wondering what licence this code is released under?

Thanks a lot for releasing this great work under MIT license!


Keeps adding tld to the entry

Whenever I need to update my domain it keeps adding the TLD to all entries in the database whenever i press Apply Changes

@Oratorian please take time to provide more information such as expected behaviors and examples. You must understand that we don't get enthusiastic about trying to provide support to a one line "bug report."

I am so sorry  ... I just remember i should provide some informations :/
I am running the powerdnsadmin\pda-legacy:latest docker container. on ubuntu
I also recieve the following error when ever i hit the Dashboard with the domain overview.
dnsadmin  | [2022-12-23 14:25:28,851] [utils.py:111] WARNING - UTF-8 content.decode failed, switching to slower .json method
I switched to the v0.3.0 in the docker container to check if this still happens there.

I did some checking, this only occurs on the

My docker somehow mixed up the old dockerhub repo with the new one and always pulled the wrong images, fixed now after a purge :/

@Oratorian no worries friend, just trying to keep things manageable with minimum resources!
Glad to hear you got the confusion worked out. I knew the Docker Hub change would cause some confusion but it's par for the course! Thank you for being a PDA user!


Feature #2054 - Add event 'lazyLoaded' so user can listen for when image loads successfully

https://jsfiddle.net/parcotk/txm44hsw/3/
i added an event that should help resolve feature #2054
Let me know if there is anything that I should change.

Thanks, @taylorKonigsmark ...
Could you do me a favour and run:
$ git rebase -i HEAD^5

on your fork and squash all your commits in to one? :) -- it's a bit messy having those revert commits :+1:

@simeydotme Sorry, still getting used to git :)
For my knowledge, we're moving the event trigger inside the animate function because that is the final step when it is loaded correctly?

correct, @taylorKonigsmark  :) -- better not fire the event before the image is visible, I think :)
don't worry about git, after a couple of times rebasing, you'll be a pro ;) -- hop in to slick on Gitter if u need help.

I think everything is set @simeydotme, does @kenwheeler need to approve?

Thanks @taylorKonigsmark ... http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html , sorry to hound you for a rebase, but it's 2 lines of code currently spread across 6 commits :cold_sweat:

holy moly what a learning curve this is! did that work? haha. Otherwise... I need to buy a book for git

🙈 Haha, so sorry @taylorKonigsmark ... I have no idea what happened, but that's definitely not what I was expecting to see 😛😘 ... I'll try to sort this out tonight, please sit tight 🙇
Thanks so much for your effort!

Thanks @simeydotme. Honestly you can scrap the PR and put it in yourself if that is easier. I'm just trying to help out where I can. I think that my problem was that I renamed a branch for this new feature. Now I learned that was a bad move ha-ha.

closed by #7abc8d9


feat(docs): Improve documentation about Search

(cherry picked from commit ec016b4abff6bdc5ae753f14cca4746944b7ed82)
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

@szalai1 @jjoyce0510 we get a lot of questions from OSS about how Search results are displayed - can we add in those details in FAQ?

The formatting of the use case list is pretty ugly - not super readable, extra * characters, some headers are italics and others not


CustomClauseQuery without a required clause?

I'm trying to query for all objects in my db for a graphql connection, is there some no-op clause I could use? Or is there another query I should use?
RIght now I'm using:
    return new CustomClauseQuery(omniVC, {
      loadEntOptions: Object.loaderOptions(),
      clause: <what do i put here>?
      name: "objects_connection",
      orderby: [
        {
          column: "created_at",
          direction: "DESC",
        },
      ],
    });

You could use Expression('1 = 1').

Or query.NotEq('id', null).

aha yes! was looking for something like Expression, thank you :)

i intentionally didn't make it easy to do table scans but yah Expression works here
The other thing you can do is Foo.loadCustom('select * from blah') but you don't get a connection for that only a list


[Site] Unequal split of image and adjoining description on main page

Description

The meshery nighthawk performance chart image is bigger than 50% of its container.
Expected Behavior

It should be adjusted to be 50% of its container.
Screenshots

Contributor Resources

See the Contributing to getnighthawk.dev Website
See the Project Site section of GetNighthawk project doc

@kayceeDev can you take this on?

@kayceeDev can you take this on?

@Neilblaze is this an issue you can address?

I don't see any such issue populating on my end, but when I zoom out to 25% (on my PC), only then, this actually become prominent. But yeah, if the CSS can be tweaked, then it can be restricted to it's 50% area-scope

@Neilblaze, good. Can you carry this forward?

@leecalcote Can you assign this to me, please?


win32 fixes

Great, merged!
Thank you for looking at the module and fixing my mistakes!
-max
Am 09.04.2016 um 22:38 schrieb Alexandr Ciornii:

    You can view, comment on, or merge this pull request online at:

https://github.com/Corion/archive-sevenzip/pull/1
    Commit Summary

Pod::Readme older than 1.0.2 may hang

skip $why, $how_many

make search for 7z faster and with less text on screen

use defined to have less warnings
File Changes

M Makefile.PL
https://github.com/Corion/archive-sevenzip/pull/1/files#diff-0 (1)

M lib/Archive/SevenZip.pm
https://github.com/Corion/archive-sevenzip/pull/1/files#diff-1 (5)

M t/20_bug_github11.t
https://github.com/Corion/archive-sevenzip/pull/1/files#diff-2 (2)
Patch Links:

https://github.com/Corion/archive-sevenzip/pull/1.patch

https://github.com/Corion/archive-sevenzip/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/Corion/archive-sevenzip/pull/1


[FEATURE REQUEST] LibRaw 0.20

Is your feature request related to a problem? Please describe.
Currently oiio doesn't build out of the box on the newly-released LibRaw 0.20 Beta 1.
Describe the solution you'd like
There are some deprecated methods, and some data types have changed. I started a patch but the changes required seem to be non-trivial
Describe alternatives you've considered
0.19.x will be EOL at some point...
Additional context
I can provide my patch so far at your request.

Which version of OIIO?

<IP_ADDRESS>.

If you can do so easily, can you just try the current master and see if that builds? I remember doing some work on this recently. If all it takes is a backport of what I already did in master, I can do that right away.

It builds, both freshly checked out and tarred up and put through the RPM build process (I'm the LibRaw Fedora RPM maintainer, for context). Do you want to backport what's needed and cut a release, or something else? I see the soname has changed.

I'll backport. Do you need a release right away?
I tested on my end as well, and it looks like I get a clean build but one of the tests fails (meaning the image is different). I'll look into this, might need a little tweaking before I backport.

It's not super urgent. Next week or whatever is fine. :)

I usually cut official tagged released on the first of each month, but if somebody really needs it, I can do any time.
I do want to track down the test failure first, though.

Cool, let me know when you've fixed the test and released something you're happy with.

Gwyn, not necessarily related to this fix, but can I ask your opinion... what's the oldest LibRaw release that you think is reasonable for us to continue supporting and testing against? (Keeping in mind that many people who build OIIO from source are relying on a LTS Linux distro of some sort, so the dependencies may be a bit old.)

EPEL for EL-6 has 0.16.x, so that's probably a good indicator.

Tagged Release-<IP_ADDRESS>, which should address this problem.

Thank you!


Index news articles

As an editor I want news articles in Sanity to be indexed so that the end users can easily find them
Acceptane criteria

The following information from a news article is searchable:

The following fields are facetable:

Ranking

Need to have a discussion about this before start working in order to get the acceptance criteria in place.

For now only slug, title and text from blocks are searchable. Each child of a block is its own record in Algolia.


[GOLANG] Better handle major version in branch name for go dependencies

In go, the major version of a dependency can be part of the namespace
e.g. github.com/shirou/gopsutil/V3
meaning the resulting branch name created by dependabot is dependabot/go_modules/.../github.com/shirou/gopsutil/v3-3.21.9
But this is not the case for all packages
e.g. github.com/spf13/viper-1.9.0 -> dependabot/go_modules/.../github.com/spf13/viper-1.9.0
The problem is that it is then quite difficult to distinguish the major version from the package name in the first case. I was hence wondering whether it was possible to express the package name with separators different from the ones used by dependabot i.e. -, _ and /, or concatenate differently the major version with the package name or remove the major version altogether since the whole version is added later on.

Hi @acabarbaye, my read of the issue is that you would like to be able to programmatically extract the package name from the Dependabot-created branches, is this correct?
If so, it might be useful to look at this action we have for extracting metadata about branches in GitHub Actions:
https://github.com/dependabot/fetch-metadata
I understand you may not analysing the branch names in GitHub Actions, but it makes use of YAML metadata that Dependabot puts in its commits, e.g https://github.com/dependabot/fetch-metadata/pull/114/commits/93245a5336d2429e548897c55a16276c70ece27e
Does this help with what you are trying to achieve?

This works well in the context of github actions. However, I am using a different CI system and the only information I have access to is the branch name

👋 Sorry for the slow response. I understand why you're asking for this, but I'm going to close this as unplanned.
I'm actually curious what CI system you're using? Because I'm really surprised you're not able to analyse the YAML metadata in the commit as described by @brrygrdn above... that feels like the most straightforward solution here.


Update the game over message

Update the game over message so people know how to play again!

Ready for review


[spi_device] Upload Cmd/Addr FIFO status revision

In previous design, the cmdfifo, addrfifo in the spid_upload module
reports the not empty status. The status bits are connected to the
UPLOAD_STATUS CSR for SW to read and check the status.
The not empty status, however, assert one clock earlier than the actual
rvalid signals.
The reason is due to the SRAM read latency. The upload module adopts
SRAM FIFO interface to maintain the cmd, addr FIFOs. When the logic in
SCK domain pushes entries, the pointer is increased. This triggers the
FIFO logic in the bus clock domain fetches data from SRAM.
The pointer crosses the clock boundary. It takes usually two cycles (in
the receiver side). Then, as the read and write pointer in the bus clock
domain has a gap by 1, the FIFO not empty status is raised.
At this time, the logic just sent the read request to the SRAM. The data
is not yet read. The rvalid asserts when it receives data from the
SRAM.
So, if the consumer reads data at the same cycle when notempty status is
raised, it reads incorrect data.
This is not a feasible scenario in the real usage as the core takes time
to read the data from the FIFO CSR.

Merging now. Thanks for reviews!


Memleak Issue with sample program using Nokogiri

Hello,
I am facing a memleak on this sample program of adding/removing nodes using nokogiri.
I have put my code at https://gist.github.com/rahulroot/dca9a06a73d9b29f7a329ae0e72001a4,
I am using Jmeter to fire the attached xml over a 100 tps. Memory grows from 14m to 3.7g in 12 hours...
First I thought there might be issue with EM, then I removed parsing logic and just bypassing request to response...and memory remains constant for that time duration....
This is my nokogiri..
nokogiri -v
Nokogiri (<IP_ADDRESS>)
---
warnings: []
nokogiri: <IP_ADDRESS>
ruby:
  version: 2.3.1
  platform: x86_64-linux
  description: ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-linux]
  engine: ruby
libxml:
  binding: extension
  source: packaged
  libxml2_path: "/tmp/tmp.2ySTDWyTqM/vendor/ruby/2.3.0/gems/nokogiri-<IP_ADDRESS>/ports/x86_64-pc-linux-gnu/libxml2/2.9.4"
  libxslt_path: "/tmp/tmp.2ySTDWyTqM/vendor/ruby/2.3.0/gems/nokogiri-<IP_ADDRESS>/ports/x86_64-pc-linux-gnu/libxslt/1.1.29"
  libxml2_patches: []
  libxslt_patches: []
  compiled: 2.9.4
  loaded: 2.9.4

It is SLES 11 sp3 VM box.
Please look.
Br,
Rahul

Thanks for reporting! I'll take a look.
On Mar 1, 2017 2:47 AM, "Rahul"<EMAIL_ADDRESS>wrote:

Hello,
I am facing a memleak on this sample program of adding/removing nodes
using nokogiri.
I have put my code at https://gist.github.com/rahulroot/
dca9a06a73d9b29f7a329ae0e72001a4,
I am using Jmeter to fire the attached xml over a 100 tps. Memory grows
from 14m to 3.7g in 12 hours...
First I thought there might be issue with EM, then I removed parsing logic
and just bypassing request to response...and memory remains constant for
that time duration....
This is my nokogiri..
nokogiri -v
Nokogiri (<IP_ADDRESS>)

warnings: []
nokogiri: <IP_ADDRESS>
ruby:
version: 2.3.1
platform: x86_64-linux
description: ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-linux]
engine: ruby
libxml:
binding: extension
source: packaged
libxml2_path: "/tmp/tmp.2ySTDWyTqM/vendor/ruby/2.3.0/gems/nokogiri-<IP_ADDRESS>/ports/x86_64-pc-linux-gnu/libxml2/2.9.4"
libxslt_path: "/tmp/tmp.2ySTDWyTqM/vendor/ruby/2.3.0/gems/nokogiri-<IP_ADDRESS>/ports/x86_64-pc-linux-gnu/libxslt/1.1.29"
libxml2_patches: []
libxslt_patches: []
compiled: 2.9.4
loaded: 2.9.4
It is SLES 11 sp3 VM box.
Please look.
Br,
Rahul
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sparklemotion/nokogiri/issues/1609, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAAgD4uAFmZyrsEJyTzybvLJH6bZOJxZks5rhSKEgaJpZM4MPUnl
.

I'm spending some time to try to reproduce this, and I don't understand some things about the code in your gist. On line 52 of your gist, this code:
         node =  Nokogiri::XML::Node.new(node_name, @document)

is suspicious. Where is @document being set? Unless there's some EM magic going on, it looks like that value should be nil, and an exception should be getting raised. Am I missing something?  Do they match what you want to see? Help me understand what you're trying to do.

Hello Mike,
My apologies, it was cut/paste. I updated, I removed the instance @document
and replace it with document.
It is updated now.
Br,
Rahul Gupta.
On Wed, Mar 1, 2017 at 7:12 PM, Mike Dalessio<EMAIL_ADDRESS>wrote:

I'm spending some time to try to reproduce this, and I don't understand
some things about the code in your gist. On line 52 of your gist, this code:
     node =  Nokogiri::XML::Node.new(node_name, @document)

is suspicious. Where is @document being set? Unless there's some EM magic
going on, it looks like that value should be nil, and an exception should
be getting raised. Am I missing something?  Do they match what you want to see?
Help me understand what you're trying to do.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sparklemotion/nokogiri/issues/1609#issuecomment-283342421,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE1luEnlacZa5EhXps0MmI-3ASI4uQHWks5rhXWwgaJpZM4MPUnl
.

Hello Mike,
I send attached xml into this program, i receive it as @http_post_content in request_handler, I use the same request as response and I parse this response in response_handler, I removed some nodes with(//pdpContext) and add some nodes with my name(rahul).
Please let me know if you have any problem in running this.
Br,
Rahul.

I have updated the code also with exception handling on namespace prefix.
Br,
Rahul.

Hello Mike,
I found the Issue in posted code, it was line number 36, if root.search throws an exception it will leak the memory....I put it under rescue and exit...I see no memleak now....
I guess next issue 1610 is valid now...
Thanks,
Rahul.


Incorporate OpenShift Functions when it GAs

Check out PR #24  for reference.  The content exists (https://github.com/RedHatGov/serverless-workshop-dashboard/blob/feature/developer-fn/workshop/content/lab1.4_developer.md), but it requires docker in docker since we use homeroom.


New post for Ideas in Progress

New post about the schools co-design activity.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@simonbates and @colinbdclark please review this PR. Thanks

We have permission to use images/StudentsActivityBoardInProgress.png?

I've created a JIRA issue for this work:
https://issues.fluidproject.org/browse/CITY-13
Please prefix commit comments for future commits for this pull request with "[CITY-13]: ".


"weareathlon/chimpjs-xfvb" downloading failed

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

Open kinematic app
Find 'chimpjs'
Start to download "weareathlon/chimpjs-xfvb". Can not be downloaded

"Tag latest not found in repository docker.io/weareathlon/chimpjs-xfvb"

Works as expected, the repo doesn't have the latest tag:
https://hub.docker.com/r/weareathlon/chimpjs-xfvb/tags/
Open an issue with the image owner


Support variables in AWS instance tag names

Thank you for opening an issue.
In this template paragraph text could be removed, however please retain headers.
Prerequisites

[x] Ensure you have latest version of plugin installed
[x] Search for possible issue duplicates

Possibly related to #303
Installation details

[GoLand 2020.2.3
Build #GO-202.7319.61, built on September 16, 2020
Licensed to christopher marget
You have a perpetual fallback license for this version
Subscription is active until August 9, 2021
Runtime version: 11.0.8+10-b944.31 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 10.15.6
GC: ParNew, ConcurrentMarkSweep
Memory: 1979M
Cores: 16
Registry: ide.completion.variant.limit=500, suggest.all.run.configurations.from.context=true, ideFeaturesTrainer.welcomeScreen.tutorialsTree=TRUE
Non-Bundled Plugins: IdeaVIM, org.intellij.plugins.hcl] IDE version (Help->About->Copy to Clipboard)
[0.7.10] intellij-hcl plugin version (Settings->Plugins)
[0.13.4] Terraform version (terraform -v)

Terraform Configuration Files
variable "foo_tag" {
  default = "foo"
}

resource "aws_instance" "example" {
  tags = {
    terraform = true
    Name      = "example instance"
    (var.foo_tag) = "xyz"                       <- this line
  }
}

Exception
Without the parenthesis the plugin is happy, but terraform complains:
If this expression is intended to be a reference, wrap it in parentheses. If
it's instead intended as a literal name containing periods, wrap it in quotes
to create a string literal.

With the parenthesis, terraform is happy, but the IDE complains:
'(', ',', <operator> or '}' expected, got '='

Expected Behavior
HCL plugin should not mark the line as syntactically incorrect.
Actual Behavior
HCL plugin marked the line as syntactically incorrect.
Steps to Reproduce
Paste example code, note red underscore at = character

As workaround for now I'd recommend to write such parameters first in object.
What happens now is plugin thinks that "example instance"(var.foo_tag) is a method invocation.

Will be fixed in next plugin release

@VladRassokhin Any ETA on a fix being released? This spurious syntax error ((...) = ... in a map) affects maybe 10% of the files I edit.

Note that 0.7.11 has still not been released, see: https://plugins.jetbrains.com/plugin/7808-hashicorp-terraform--hcl-language-support/versions
Also, I wonder if this is the right place to raise issues on the plugin, since JetBrains have their own YouTrack issue tracker, and that also contains issues related to the Terraform plugin; e.g.: https://youtrack.jetbrains.com/issue/IDEA-250289


Update test-file-04.md

#sign-off

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

csidevdocs-public/Articles/test-file-04.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off

@v-caxian: Only the author, @TinaLiuGirl, can sign off on this pull request.

@v-caxian : Thanks for your contribution to the Azure documentation! The author, @TinaLiuGirl, has been notified to review your proposed change.

@v-caxian: Only the author, @TinaLiuGirl, can sign off on this pull request.

@v-caxian : Thanks for your contribution to the Azure documentation! The author, @TinaLiuGirl, has been notified to review your proposed change.


'p.'/'pp.' abbreviation retained with Roman numerals

When using a page number with Roman numerals, the abbreviation is not suppressed in version 0.7.4 (compare Chicago Manual of Style 14.154):
pandoc -F pandoc-citeproc << EOT
---
references:
- id: test
...

[@test, p. ix]

[@test, pp. ix–x]

EOT

Actual output:

(n.d., p. ix)

(n.d., pp. ix–x)

n.d.

Expected output:

(n.d., ix)

(n.d., ix–x)

n.d.

This is because of the current logic for parsing locators out of the generic suffix.
What we get from pandoc is just a suffix -- the part after the citation key itself -- and pandoc-citeproc needs to separate this into a proper locator and a remainder (the suffix).  The heuristic we use for identifying locators is designed to capture locators of all of the following forms:
123, 123A, C22, XVII, 33-44, 22-33; 22-11

What is conspicuously missing are lower-case roman numerals.  Why?  Because it was hard to find a simple heuristic for recognizing these that wouldn't capture regular text, too.  But it should be possible to come up with something a bit more robust that could allow lowercase roman too.  The relevant part of the code, for reference, is pLocator and pWordWithDigits in src/Text/CSL/Pandoc.hs.

To give a better idea of the difficulty:
@smith37, pp. 123A, C22, VII, 22-33, and elsewhere

here we want the locator to include "123A, C22, VII, 22-33"; the rest, , and elsewhere, is the suffix.  So we have to be able to determine where the locator stops.  Currently that's done by looking for words that contain arabic digits or uppercase roman numerals....a very rough heuristic.  I'm open to suggestion for something more robust.

The present behaviour allows it to parse things such as [@test, p. 123n5] properly (and technically one could likewise have [@test, p. liin4] in Chicago Style), so I wouldn't change its treatment of words with Arabic numerals.
There's some good regex examples for detecting Roman numerals, but only for uppercase (apologies if you're already using something like this, as I haven't found the code, but it appears to be something similar to their 'flexible' rather than 'strict' example). I suppose this could be extended by looking for words that match this pattern and are entirely uppercase or lowercase (I don't think it's possible to have a mixed-case Roman numeral). I can't presently think of a situation in which this treatment would cause a problem.

+++ Andrew Dunning [Sep 26 15 19:39 ]:

The present behaviour allows it to parse things such as [@test, p.
123n5] properly (and technically one could likewise have [@test, p.
liin4] in Chicago Style), so I wouldn't change its treatment of words
with Arabic numerals.
There's some good [1]regex examples for detecting Roman numerals, but
only for uppercase (apologies if you're already using something like
this, as I haven't found the code, but it appears to be something
similar to their 'flexible' rather than 'strict' example). I suppose
this could be extended by looking for words that match this pattern and
are entirely uppercase or lowercase (I don't think it's possible to
have a mixed-case Roman numeral). I can't presently think of a
situation in which this treatment would cause a problem.

I was thinking there might also be cases with mixed roman
numerals and, say, letters:  ixn5, for example.  So it gets
really complicated to detect these reliably.

Unless I am missing something, that's not a problem if you're already treating any word including a number as a locator. The example I gave above of [@test, p. liin4] already works, as does something like p. 2:ix.

So, is your suggestion to look for (a) words containing
digits, and (b) words that can be wholly interpreted as roman
numerals (excluding punctuation)?
+++ Andrew Dunning [Sep 27 15 10:59 ]:

Unless I am missing something, that's not a problem if you're already
treating any word including a number as a locator. The example I gave
above of [@test, p. liin4] already works, as does something like p.
2:ix.
—
Reply to this email directly or [1]view it on GitHub.
References

https://github.com/jgm/pandoc-citeproc/issues/173#issuecomment-143581153

PS.  there's already a romanNumeral parser in Text.Pandoc.Parsing.
+++ Andrew Dunning [Sep 27 15 10:59 ]:

Unless I am missing something, that's not a problem if you're already
treating any word including a number as a locator. The example I gave
above of [@test, p. liin4] already works, as does something like p.
2:ix.
—
Reply to this email directly or [1]view it on GitHub.
References

https://github.com/jgm/pandoc-citeproc/issues/173#issuecomment-143581153

Right, (a) being what I understand to be included in the present behaviour. Interpreting entirely lowercase words that can be parsed as Roman numerals might exclude a few edge cases (e.g. a § vii A?); but it would cover the main usage of citing front matter, and adding only this rule shouldn't introduce any adverse effects.

Wonderful; thank you!


fix linter packages

Replaced eslint config pkg with airbnb-base since we're not using React.

Coverage remained the same at 100.0% when pulling 45c1cc3f1f10acd095fa3250fe2497dc4968eb25 on fix-eslint-pkg into 0b5f023f0acc3754b04a5575a1fc2321a7a713c0 on develop.


Update research page

The changes I made all look fine but I notice - seem to be changed to &#8211which appears to be the Unicode Decimal Code. Is this on purpose? Does it matter?

Not on purpose, and doesn't matter.  Probably just an artifact of how I copied/pasted the HTML from the blog.


Report issues after testcase failure_ TestNG Testing Methods

Code covers:

Reporting bugs for TestNG test methods failure with the following data:

Attachments [per test Method].
Test Method log in the bug description.
The bug will be named after the test-case method name.
Linked to the test case only if TMSLink is provided.
assignee if provided in the configuration, the issue will be not assigned if not.

configuration manager updated with the required enablers.

Notes:

Reporting issues can be only enabled by enabling the integration and report bugs options from the configuration manager.
Reporting bugs is not linked to the execution reporting. which means that the user still has the flexibility to report bugs without reporting the execution.
It is pure JIRA integration doesn't depend on any TMS plugin.

Next: Cucumber scenarios bugs reporting.

Please and Thank You.

Privacy Issues Data Loss

Please AndThank You

please and thank  you


fix: Improve chunked upload reliability (box/box-codegen#529)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
10 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 70.37%

Files with Coverage Reduction
New Missed Lines
%

Box.Sdk.Gen/Managers/ChunkedUploads/GetFileUploadSessionByIdHeaders.cs
4
0.0%

Box.Sdk.Gen/Managers/ChunkedUploads/ChunkedUploadsManager.cs
6
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
6671

Relevant Lines:
9486

💛  - Coveralls


How do I monitor load and scale?

Hi, I asked this question on one of the examples @drtriumph provided.
I think I'd use the firebase-queue to handle incoming messages (like the sanichat example). Except I want them to get handled after exactly 60 seconds from when they are "published" (from that example). So as long as a worker gets to it within the 60 seconds it can run a timeout for the time left, process it, and resolve it.
But, I'm wondering how to know when is too many workers for the cpu. Because that's the number of workers I'd use.
Then, I'd set the cpu threshold to something less than that - when it gets reached, Azure will add another vm with the same number of workers...
Any ideas? Go Firebase!
(Label this a question of course)

This is a great question, and my initial answer is that there's not a whole lot of instrumentation for this use-case currently. I do, however see great value in this, so I'll have a think about how more can be built-in to the queue itself.
I think in your position I would start with monitoring the number of outstanding queue items, and the timeout lengths with some stats. As the timeout lengths creep down towards 0 you will be seeing the saturation of your pipeline, though that's not directly able to be automated. Timeouts in Node are not of course guaranteed to fire at the exact moment you schedule them for, so I'd also monitor the actual length of time between the item being added to the queue and it being processed.
Under load I'd then experiment with the number of workers. I usually suggest a single worker per CPU, but given the process is likely to just be waiting for a while, you probably want more. How many more will depend on how long your processing function usually takes - if it takes 5 seconds you might want to start with something like 10 workers per CPU.
Once you've settled on a number of processes per CPU, under load you will probably be able to come across some kind of heuristic for the number of outstanding requests per machine and then scale on that.

hi @drtriumph,
can you give us an example please?


Scheduled / One Time: "Discharge home battery to Car"

Is your feature request related to a problem? Please describe.
If you want to discharge you home battery to your car you have to manual play around with max discharge settings and "schnell" mode
Describe the solution you'd like
Loading Mode "Discharge Home Battery" will drain home battery (max discharge power and low SoC configurable) to the car.
Example: This morning i had 30% (~3kWh) left in my Huawei home battery storage and i want to charge my car with it before heading to work. During work hours my home batterie will charge up to 100% again.
To drain the home batterie to my car i manual have to set the max amp to 7 (max discharge of home battery is 5kW) and select "Schnell".
At 3% SoC of the home battery i had to manually stop the charging process.
I guess their is something more automation possible.
Additional context
In combination with PV yield prediction (e.g. https://forecast.solar/) this can be used for a new charging plan option.
https://forecast.solar/ is also used by HomeAssistant PV yield prediction and at least for my home is some kind reliable.
theoretically scenario:
At some point (evening or early morning) i have xx % SoC (configurable) in my home battery. The PV yield prediction provides information that on the next/same day the pv yield is higher than yy kWh (configurable). If this is true, that discharge home battery to car.
Example for me: At 5am: If Home Battery is >30% and PV yield prediction is > 25kWh then discharge home battery to car until home battery is <10% SoC
Personal Question: GitHub Issues in English or German?

Sound similar to #12190

Yes, general idea is identical. Merge if possible.

Then let's move the discussion to https://github.com/evcc-io/evcc/discussions/12190


transition to fido2 1.0.0 and higher

Discussed in https://github.com/solokeys/solo1-cli/discussions/156

Originally posted by vladak June 23, 2022
solo1-cli version 0.1.1 (with changes matching PR #153), and brand new Solokey with updated firmware:
$ solo1 ls
:: Solos
None: SoloKeys Solo 4.1.5

and it is not possible to change the default PIN:
$ solo1 key change-pin
Please enter old pin:
Please enter new pin:
Please confirm new pin:
'Fido2Client' object has no attribute 'client_pin'

$ solo1 key set-pin
Please enter new pin:
Please confirm new pin:
'Fido2Client' object has no attribute 'client_pin'

I installed the solo1 via the usual means (Python venv + pip install).
The fido2 package is at version 1.0.0.

This is not the only problem with 1.0.0 (see the related discussion), so I changed the bug synopsis.

Possible fixes in https://github.com/trustcrypto/onlykey-solo-python/commit/7e570dc0f7292eded4644f4eb637e22308aebd91 (files solo/devices/base.py, solo/devices/solo_v1.py, solo/solotool.py)

Possible fixes in trustcrypto@7e570dc (files solo/devices/base.py, solo/devices/solo_v1.py, solo/solotool.py)

There are not, this is just a subset of #159 and pretty poorly done (removing some files and adding some others without a reason).

@nickray @kms15 any update on the fido2 >= 1.0 transition. Debian would like to get fido2 1.1 in and solo1-cli is currently blocking.

I started working on this not in PR #169 - it's very much WIP. Please test & report breakages as I don't really have an idea how to systematically test ...

@nickray If you don't act now, this will kick solo1-cli out of Debian, Ubuntu and all their downstream distros unfortunately.

...will kick solo1-cli out of Debian, Ubuntu and all their downstream distros unfortunately.

The same applies for NIxpkgs.

@fabaff You could use PR #169. This version is in Debian testing for six months now without bug reports.


Fix {{this.foo}} !== {{foo}}

The handlebars parser simply drops the this for all path expressions, but this is incorrect in Glimmer since we have the concept of local variables (block params), so {{this.foo}} is semantically different from {{foo}} (the former is always a “self lookup”, whereas the latter might be a local variable lookup).
This fundamentally fixes the problem by adding this back into the path expressions and allow the runtime to triage it. This also removes the need for the “self-get” expression we previously added to address this issue.

LGTM :+1:


Layer Panel: Circle shape not looking circular

Bug Description
Circle shape preview in layer panel gets cut off.
Expected Behaviour
Circle shape preview in layer panel should look circular.
Steps to Reproduce

Add circle shape to story
Look at layer panel

Screenshots

Additional Context

Plugin Version: 1.14.0-alpha
WordPress Version: 5.8.1
Operating System: Mac OSX BigSur
Browser: Chrome

Verified in QA


Bumping RSpec to 3.3

I have a PR that I'm writing that would require RSpec 3.3 [1], so I'm requesting this bump.
RSpec 3.3 was released one month ago (3.2 Release Notes and 3.3 Release Notes). bundle exec rspec works out of the box for me in scss-lint with RSpec 3.3.0 and Ruby 2.0.0.

[1] I'm adding some tests for report_lint which would use fail_with, etc. from rspec-expectations. From the Release Notes:

"Add RSpec::Matchers::FailMatchers, a mixin which provides fail, fail_with and fail_including matchers for use in specifying that an expectation fails for use by extension/plugin authors. (Charlie Rudolph, #729)"

Merged in ce7edf0a694cb56d85d3a3ef2e4c9551aac7ec61.


The community 'swarmbot-lovers' does not exist.

Check full details on Airbrake: #1556419794276306975
Backtrace:
/app/node_modules/bluebird/js/release/errors.js:74:in Function.OperationalError
/app/src/models/dco.coffee:29:in
/app/node_modules/bluebird/js/release/util.js:11:in tryCatcher
/app/node_modules/bluebird/js/release/promise.js:488:in Promise._settlePromiseFromHandler
/app/node_modules/bluebird/js/release/promise.js:545:in Promise._settlePromise
/app/node_modules/bluebird/js/release/promise.js:582:in Promise._settlePromiseCtx
/app/node_modules/bluebird/js/release/async.js:130:in Async._drainQueue
/app/node_modules/bluebird/js/release/async.js:135:in Async._drainQueues
/app/node_modules/bluebird/js/release/async.js:16:in Immediate.Async.drainQueues [as _onImmediate]
timers.js:383:in processImmediate [as _immediateCallback]

Gets stuck as well.