getNativeOrientation() on iOS returns the current device orientation, not the native one

On iOS backend ( see https://git.io/JEFzU ) getNativeOrientation() returns the current device orientation, not the native one. So when you handle device rotations (e.g. within the ApplicationAdapter's resize method), you cannot know exactly what situation the device is in. BTW, on Android getNativeOrientation() always report the correct value (i.e. the same value independently from the current device rotation).
I would suggest a tricky code, in order to precalculate (on iOS) the native orientation:
int screenWidth = Gdx.graphics.getBackBufferWidth();
int screenHeight = Gdx.graphics.getBackBufferHeight();
int deviceRotation = Gdx.input.getRotation();

if (((deviceRotation == 0  deviceRotation == 180) && (screenWidth >= screenHeight))
    ((deviceRotation == 90 || deviceRotation == 270) && (screenWidth <= screenHeight))) {
    _nativeDeviceOrientation = Input.Orientation.Landscape;
}
else {
    _nativeDeviceOrientation = Input.Orientation.Portrait;
}

Feedback would be appreciated. Thanks for your hard work!

On iOS getNativeOrientation returns Portrait if the current rotation (i.e. the value returned by getRotation) is 90 or 270, else it returns Landscape. So, imagine to handle device rotation within the resize method: how can I handle the event correctly if:

I can get the current rotation using getRotation()
But I cannot know the native device orientation

For example, lets say getRotation has returned 90. How can I know if the device is a native portrait device being rotated by 90 degrees or if the device is a portrait device being rotated by 90 degrees? In the general case the behavior that the application must have can be different.
Thanks!

You don't need to worry about how the device is normally held (the "natural or native orientation"), the iOS API tells you how it's being currently held that's what you want to know.
If you check getRotation() implementation, iOS doesn't return an angle but an orientation and on iOS "Portrait" always means width < height and "Landscape" width > height (it's also how we generally define portrait and landscape when we talk). So, following your example, if getRotation() has returned 90 you know for sure the device is in Landscape and that width > height.


remove some backports

We shouldn't need these anymore.

:+1:

Gotta love code that screams to be removed.


⚠ Bofrost.be has degraded performance

In f76f5e6, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1200 ms

Resolved: Bofrost.be performance has improved in 88fe51b after 18 minutes.


[Bug]: Not able to adjust height of TreeSelect widget

Is there an existing issue for this?

[X] I have searched the existing issues

Description
I am trying to increase the number of rows shown in a treeselect widget.
However, adjusting the 'Height' setting does not appear to have any effect. The number of rows displayed remains at 5 no matter which option is selected or how the height is adjusted.
See loom video for an example.
Steps To Reproduce
Create a TreeSelect widget, add some content and try to adjust the height.
Public Sample App
No response
Environment
Production
Severity
Medium (Frustrating UX)
Issue video log
https://www.loom.com/share/04134ece45164dea97b1d97953accbec?sid=7213942b-fa51-4058-85b3-f7141e1cd9aa
Version
Cloud

@samiphoenix84 the treeselect should not have a height option. We'll remove it.
@rahulbarwal

Hey @Nikhil-Nandagopal  will someone work on this issue or can I work on it ? if you guys are planning to implement this feature?

Hi @ALOK9442. Thank you for your interest in contributing. We have marked some issues with Inviting contribution label we would appreciate it if you could take a look at those, if possible.
Feel free to use this link: https://github.com/appsmithorg/appsmith/issues?q=is:open+is:issue+label:"Inviting+Contribution"

Hello @samiphoenix84, thank you for reporting this. Could we have a brief call to discuss your use case? I'd love to understand it better. If you're available, feel free to book a time that works for you here


[🐛 BUG]: RR cannot parse larger multipart form data

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

What happened?
RoadRunner is unable to parse larger multipart form data.
In my case the whole POST body is around 260KB whoch is not really a lot and contains a moderate amount of inputs. RoadRunner throws error 500 with message serve_http: multipart: message too large.
The current workaround is to send JSON instead, but mapping FormData to JSON and back can be prone to errors.
Version (rr --version)
2023.3.9
How to reproduce the issue?
curl.txt
Relevant log output
No response

This has not been solved for me

Not confirmed as a RR issue. Thread: https://discord.com/channels/538114875570913290/1229393196648366201

I'm having the same issue. When I try to submit a request with multiple array input fields that are over 256KB, I get serve_http: multipart: message too large

It's a bug in Golang I presume. If you are not sending files, use either application/x-www-form-urlencoded in body instead or send json, if you can.

Thanks! That worked.


fix(AsyncSubject): do not allow change value after complete

Description:
This PR updates AsyncSubject::next adds guard to now allow to change value if it's already completed.
Related issue (if exists):
closes #1800

Coverage increased (+0.0005%) to 96.472% when pulling e73cb62e1833741b9939ba245986249ce7f0496b on kwonoj:asyncsubject-next into 5158f27456ca5a7fac14f706e7df8ffd4676789b on ReactiveX:master.

Coverage increased (+0.0005%) to 96.472% when pulling f8391d455db257f8ee3606917b67caf71c6ccb29 on kwonoj:asyncsubject-next into 22fd8aeeb4b30038626f4274662cdbffab770067 on ReactiveX:master.

Coverage increased (+0.0005%) to 96.472% when pulling 6db63caa1d1c6c7c69e425c8740ff9cc110b113b on kwonoj:asyncsubject-next into 8befc1e2e3cfc0364db8190a7b8b093da3feec19 on ReactiveX:master.

Coverage increased (+0.0005%) to 96.472% when pulling 8193b19e94c091a89cc2b73ef217b7d012343f11 on kwonoj:asyncsubject-next into 0fc88b2807cf0073b1e511ef6f2a5c89d700a35b on ReactiveX:master.

LGTM

LGTM

Coverage increased (+0.0005%) to 96.472% when pulling 44bf637c1ed8071b4c5ed8bfb5de6cbeea405307 on kwonoj:asyncsubject-next into 3632489b37d04a99746dfb85a9ae34173cbc8fb2 on ReactiveX:master.

Coverage increased (+0.0005%) to 96.472% when pulling 5e40c716436936d1ba455b8b208ebcc860c92cac on kwonoj:asyncsubject-next into<PHONE_NUMBER>4a2846916696483c8165a82aaae44e on ReactiveX:master.

Coverage increased (+0.0005%) to 96.472% when pulling 6aacc1f6ab68afb280629e13c10a28696bf2896c on kwonoj:asyncsubject-next into 19f43629be34c80db8c2850e01bb09b4b93b3930 on ReactiveX:master.

Coverage increased (+0.0005%) to 96.471% when pulling 6282e1165c6443c5849d1389c2b73888599f0fe3 on kwonoj:asyncsubject-next into ffcb9fcf0ebb1ad67b7ed0d1dac69ee37c688da5 on ReactiveX:master.

@kwonoj needs a rebase.

Coverage increased (+0.0005%) to 96.473% when pulling a42ca79b5eea0bee73900fe59046d41603bb2151 on kwonoj:asyncsubject-next into c76a14d4e236139076db94086648bdbce88b2e15 on ReactiveX:master.

@blesh rebased.

LGTM, needs rebase. ping me.

Coverage increased (+0.0005%) to 96.7% when pulling 801f28218e9812a5643e2e872de61b9f8b22a7ae on kwonoj:asyncsubject-next into 75a355f7d84ac65aab715130172451080ec92597 on ReactiveX:master.

@jayphelps , thanks, rebased. Have a nice weekend.


Update HelloSign integration to Dropbox Sign

HelloSign has become Dropbox Sign, so we need to update the documentation (logo, name, screenshot) and review the integration. See: https://sign.dropbox.com/blog/hellosign-now-dropbox-sign

CZO thread

@alya Can you please describe the issue ? Do you want to just update the text, logos and screenshots or also the function names present with the text "HelloSign" ?

@zulipbot claim

@zulipbot unassign

@zulipbot claim

@Niloth-p assigning to you, since there hasn't been active work here.

@zulipbot claim

@zulipbot claim


How to setup own page titile in Viewer?

Hi. When load pdf in viewer he takes page title from pdf name. But i need set up by own title. Is here some oprion to setup it like - var title = 'own title'; ?

Hi. You can change the logic inside setTitle method at https://github.com/mozilla/pdf.js/blob/master/web/viewer.js#L459. Closing as answered.


Rolebanlist command UI

About the PR

The command rolebanlist opens the UI ban list window as well as outputs a list in the console. Follows the similar style of the banlist command.
Why / Balance

This change addresses the issue #29443
While this doesn't effect the game play in anyway, it creates some uniformity in command outputs.
Technical details

The rolebanlist command makes a call to open the banlisteui in the same style as the banlist command.
Media

Requirements

[ X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[ X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

(no changelog necessary.)

can you make it open ui on the second with rolebans?

I'll look into a solution for opening the ui with rolebans tab active.


Check if failed log storage is disabled in Queue Service Provider

Fixes this issue: https://github.com/jenssegers/laravel-mongodb/issues/2356

You're right I hadn't spotted that in the original Laravel code

@divine updated that now

This won't work as $config is undefined.

Thanks!


Accessing url has redirect loop

I visited some website. Some working normal, and the other has redirect loop. I've search some forum and i've no answer yet.
http://<IP_ADDRESS>/portal-journal/
(Proquest and Scopus link. This webpage has a redirect loop).
can you please take a look?
thank
#sory my bad english

https://github.com/Athlon1600/php-proxy/issues/22


Error with downloading weights

Hi, there's something wrong when I download the weights, and I've checked my connection is fine. Is there anything wrong with the weights source? The url can't be opened directly, could you please kindly provide a available address to download the weights?
Here is my error when run pytest:
raise ConnectionError(e, request=request)
E           requests.exceptions.ConnectionError: HTTPSConnectionPool(host='docs.google.com', port=443): Max retries exceeded with url: /uc?export=download&id=1CwChAYxJo3mON6rcvXsl82FMSKj82vxF (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f33d979da58>: Failed to establish a new connection: [Errno 101] Network is unreachable',))
../miniconda3/envs/anonyai/lib/python3.6/site-packages/requests/adapters.py:516: ConnectionError
--------------------------------------------------------------------------------------- Captured stdout call ----------------------------------------------------------------------------------------
Downloading face weights to /tmp/pytest-of-root/pytest-1/test_it_downloads_weights0/weights/weights_face_v1.0.0.pb
Downloading 1CwChAYxJo3mON6rcvXsl82FMSKj82vxF into /tmp/pytest-of-root/pytest-1/test_it_downloads_weights0/weights/weights_face_v1.0.0.pb...

We fixed an issue with the download of the weights. Automatic download should work again and additionally, I provided the links at
https://github.com/understand-ai/anonymizer/blob/master/README.md#weights


Add an Edit button to the Documentation subsite

Each page of the Documentation subsite could have an edit field in its header
---
layout: docs
title:  "Examples"
edit: https://github.com/effekt-lang/effekt-website/edit/master/examples.md
---

which renders as some [✏] button somewhere in order to quickly edit the correct file in the correct repository.
If I recall correctly, GitHub should offer you to create a fork with your changes followed by a PR.
This way, it's easy enough to fix typos/problems on all documentation pages: in case studies, in the upcoming Tour de Effekt, etc.
Scala's documentation has a very similar button:

Related to #80: we could not only have an "Edit" button, but also a "Link / Go to Source" button -- that would already help (and it's mostly the same code as this issue!)


Camera Direction Not Saving

After I make a camera point, the zoom in amount seems to work fine, but the direction (north, south, east, west) doesnt save. The first couple times pressing the hotkey for the camera point it will remember, but a minute later the directions wont work, then when i go to check the side-bar app, the camera points are back to "unchanged".

Turns out the direction was saved correctly but it was never loaded when loading the camera points. So once this is fixed, they should all appear like they have been saved previously.
Thanks for letting me know!


Allow non ts/es styles

Hi there, thanks for the great module

Hi @jquense
good idea, thank you! – I just released a new minor version of import-sort (and related packages).


"Reproduce this error" instructions are not escaped correctly

When catkin_make encounters an error, it does not print the error message it encountered (doing so would be very hard to read), but instead it instructs the user how to reproduce the error:
Reproduce this error by running:
==> cd /Users/jjclark/ros_catkin_ws/build_isolated/asmo && /Users/jjclark/ros_catkin_ws/install_isolated/env.sh cmake /Users/jjclark/ros_catkin_ws/src/asmo_ros -DCATKIN_DEVEL_PREFIX=/Users/jjclark/ros_catkin_ws/devel_isolated/asmo -DCMAKE_INSTALL_PREFIX=/Users/jjclark/ros_catkin_ws/install_isolated -DCMAKE_BUILD_TYPE=Release -G Unix Makefiles

However, the printed command is not escaped correctly to support running without modification. In this example, the argument Unix Makefiles is not quoted. Any special characters in directory names would also cause problems. So when the user tries to run this command and see the error, they will see a different error instead.
I propose using pipes.quote() to escape this command so that it can be copied and pasted to produce the expected result.

Thank you for the PR. I will add a few comment inline.

Thank you for the review. I have made the suggested changes and squashed them into one commit.

I don't believe there are any other places where commands need to be quoted. Most situations where commands are printed are as informational displays, only this situation is meant to be passed through a text-only pipe (copy/paste).

The patch looks good and will help users to reproduce the error. Thanks!
I have manually merged it in ff32bf9ce09481394bae93da25249268f7e7d15f with slight changes to the comments and order of imports.


Update iOS SDK in 3.x maintenance branch

This is a partner PR to the Android version of this PR: https://github.com/BranchMetrics/cordova-ionic-phonegap-branch-deep-linking-attribution/pull/631

Update to iOS 0.31.4 in the 3.x maintenance release line
Updates plugin.xml with new framework files & dependencies
Adds new functions from new SDK version
Removes old SDK functions (stubs JS invocations to not break downstream consumers)

@bklastaitis-branch @echo-branch I have an outstanding question. Should I bring in the change that removes the duplicated entitlement files as well? The purpose of the 4.x releases is the move to CocoaPods, but the entitlement files issue isn't relevant to that - it has to do with the Cordova version.
Cordova 8's behavior is to merge the entitlement files
Cordova 9's behavior is to only use the default entitlement.plist file
The updated behavior then should work on both platforms (Cordova 8 will work with a singular entitlement file as well); the problem is users who have already installed prior versions will have a plist file that expects the split entitlements. They'd have to delete their platforms and start from zero. This issue wouldn't affect CI builds.
What would be the best course of action here? I know some folks have asked us to backport that change.

@echo-branch The plugin type and version are set in this PR, I know it's hard to find thanks to updating the 125 SDK files. See here:
https://github.com/BranchMetrics/cordova-ionic-phonegap-branch-deep-linking-attribution/pull/632/files#diff-558d913b4a56f6e9e18b183c951a290fR82

Looks good.  I'm going to merge it in and prep a release.


[Compatibility] Module does not work in Shadowdark system

I figured this module would be system neutral, but unfortunately it does not seem to work in the Shadowdark system. The scene stays blank when the module is enabled. I only have Zoom / Pan options and libWrapper enabled.

I think this was actually solved by 1.13.1 and was caused by the missing default settings.

Sweet!


Teacher-Free Adaptation is Latent Consistency Fine-tuning (LCF)?

Great works~
I am confused about 'Teacher-Free Adaptation'. Does it mean Latent Consistency Fine-tuning (LCF)? Directly select two time steps and get nosied data z_{t} and z_{t-1}, and then directly calculate the consistency loss for these two time steps to enforce self-consistency property as LCF in LCM paper?
So the training procedure is ：

Train base image diffusion model, using Latent Consistency Distillation and image data.
Fix the lcm image diffusion weight and add a trainable temporal layer, using Latent Consistency Distillation and new Initialization strategy and video data.
Add other controls (controlnet or IPadpter), using Latent Consistency Fine-tuning and the data about control conditions.

Do I have an exact understanding?

Good summary! Thanks for your interest and careful reading. For the Teacher-Free Adaptation and Latent Consistency Fine-tuning, I would say that they are technically the same thing, all following the consistency-training (CT) proposed in consistency models by Song et al. The difference is that we verify that it works for training  adapters from scratch or fine-tuning existing adapters to follow consistency property, mainly at the purpose level.


Add flag to Search2 api, indicating if search term was found in item attachments

Add flag to Search2 api, indicating if search term was found in item attachments

Checklist

[x] the contributor license agreement is signed
[x] commit message follows commit guidelines

Description of change

#1306
This was not as straight forward as originally envisioned. SearchResource.scala was previously performing a search using the searchIds method in FreeTextService
as opposed to
search method in FreeTextService,  which returns an FreetextResult object with the keywordFoundInAttachment data required

Test is at https://github.com/openequella/openEQUELLA/blob/component/new-search-page/autotest/OldTests/src/test/java/io/github/openequella/rest/Search2ApiTest.java


Creating Benchmarking Setup to compare individual sorting algorithms

Added gitignore file
Added a VS2017 solution
Added a console application that uses BenchMarkDotNet to benchmark individual sorting algorithms
Moved BubbleSort example
Added two LINQ sort styles

This PR will be closed because it has many files. At max 5 algorithms per PR.
It should only include C# files (*.cs)


Tasks are still in IN_PROGRESS/SCHEDULED state even though workflow/sub-workflow are in FAILED/TERMINATED state

Hi,
We are running Conductor v2.26.0 with Elasticache Redis cluster as our metadata store and Elasticsearch 6.x as our indexing store.
We also leverage dynamic forks and sub-workflows heavily in our project. We notice that when sub-workflows fail, they don't actually put their tasks in a CANCELLED or a FAILED state. Rather they are still SCHEDULED/IN_PROGRESS.
Because of this, our workers tend to poll tasks from workflows that are already in an end state unnecessarily.
When they are polled and updated by our workers, we see this in the Conductor server logs:
[2021-03-08T05:43:01.840Z] INFO qtp652481263-19664 WorkflowExecutor:799 Workflow: <REDACTED SUB-WORKFLOW NAME>.1/3dce8785-d71f-45f2-9f35-7ca4bef39082.TERMINATED has already finished execution. Task update for: 8c154c7d-3be2-48c3-bfa0-227dd9370946 ignored and removed from Queue: <REDACTED TASK NAME>.

Is there any solution/fix for this?

hi @bharadwajrembar, thanks for reporting this. We have also identified some glitches with subworkflows inside a fork join. There is a fix on the way for 3.0 which will be back-porting to 2.30. I am marking this as a bug and will update this thread when the fix is released.

@aravindanr Hi, is there any progress on the fix? Either in 3.0 or as a back port? It appears that we might be encountering this bug as well.

This should be fixed as part of #2125 in v3.x
Given the expanse of the changes, it was decided to not backport to 2.31 at the time. However, if you have the bandwidth we would welcome a PR with the changes.


Added V1_TAG_FILTER environment variable

Added tag filter so a user can filter on tags as well as repos
Added $AWS_REGION env variable so the script can run without input, will break tests

I was finding at $AWS_REGION was requiring me to specify a region, and I needed the script to run without user input

@Brett55 - sorry for the delay, LGTM!  Thanks!


nest take over

Taking over nest. @kedean, please revert this when you feel like maintaining your lib.

You are taking it because you the author didn't respond within less than 24 hours? What if they had other stuff going on/hadn't had time to review your PR yet? This feels premature.

@neeasade, true, normally I would wait a bit longer, but (a) given that the lib doesn't compile because of changes in Nim that happened a couple of releases ago I would assume that @kedean is no longer interested in support, (b) judging from his github activity I would assume the same again, and (c) this particular PR can always be reverted with a single click and honestly I would be happy in case it is reverted.

Resolved.


Add the DoubleWidth type

This is an experimental library for issue apple/swift-numerics#4.
The new module isn't re-exported by the Numerics module, so it isn't part of the public API.
I'd also like to implement a SIMD-based MultipleWidth type (in another pull request, if this one is accepted).
I'm not sure how to add a library product to the CMake build.

The DoubleWidth.swift file is generated from the latest GYB template:
utils/gyb --line-directive="" test/Prototypes/DoubleWidth.swift.gyb

arch
config
compilation

x86_64
debug
1.2 seconds

x86_64
release
2.5 seconds

arm64
release
2.4 seconds

The DoubleWidthTests.swift file is a translation of the original tests (from StdlibUnittest to XCTest), excluding the expectCrashLater() calls.

arch
config
compilation

x86_64
debug
5.8 seconds

x86_64
release
9.7 seconds

arm64
release
9.6 seconds

There are also some new and updated tests — for string conversions and SR-6947.

arch
config
all / slowest tests
execution

x86_64
debug
all DoubleWidthTests
2.0 seconds

x86_64
debug
…_Hexadecimal()
1.2 seconds

x86_64
debug
…_LeftAndRightShifts()
0.7 seconds

x86_64
release
all DoubleWidthTests
1.2 seconds

x86_64
release
…_Hexadecimal()
0.7 seconds

x86_64
release
…_LeftAndRightShifts()
0.4 seconds

I tried to test the DoubleWidth: _ExpressibleByBuiltinIntegerLiteral conformance:
+      swiftSettings: [.unsafeFlags(["-parse-stdlib"])]

+import Swift

-  public init(_builtinIntegerLiteral _x: Builtin.IntegerLiteral) {
+  public init(_builtinIntegerLiteral _x: Builtin.IntLiteral) {

-    if !Bool(_overflow) {
+    if !Bool(_builtinBooleanLiteral: _overflow) {

-  public init(integerLiteral x: Int) {
-    self.init(x)
-  }

There are two unrecognized builtins:

Builtin.sext_Int64_IntLiteral
Builtin.ashr_IntLiteral

When arbitrary-precision literals were implemented, DoubleWidth: _ExpressibleByBuiltinIntegerLiteral was already within the #if false block:
https://github.com/apple/swift/pull/20208/files?file-filters[]=.gyb#diff-11720e84c8f7b8f4dfedfb52e97e620ec4d9249f662995d7fc67c6ce96c0438f

Hi @benrimmington thanks. Just to set expectations, I'm booked up this week and on vacation next week, so it will probably be a little while before I get to take a look at this in any detail.


ora2pg integration

Tested and validated that this works.  What a great extension!  In the past I used a snippet of SQL code that I found to install to simulate a GTT.
So this gets me to thinking about ora2pg... Right now ora2pg does not support extraction of GTTs from oracle since PG doesn't support them.  But with this extension and if a boolean is turned on in parms to ora2pg, enable_GTT, the GTTs would be downloaded and assuming the target PG database has gtt extension created and loaded, I wonder if we could add the feature to migrate GTTs?
I haven't thought it all through yet, but whatcha think?

Yes this is the objective of this extension to allow Ora2Pg to migrate Oracle GTT. I'm adding an issue to Ora2Pg to not forget :-) Many thanks for the tests Michael and for the confirmation that it works.


No .closePopup() method

Hello. Thank you for this amazing library.
I've got a problem with closing popups.
When I bind a popup to marker there's no default closing button, the only way to close it is to click somewhere on the map.
I managed to add button by myself using Awesome Fonts and some closing method.
var popup = L.popup()

    .setLatLng(latlng)
    .setContent('<i class="fa fa-times icon-popup" onclick="closePopup()">        </i><br> My content ');

    marker.bindPopup(popup);
    popup.addTo(layer);

I've tried almost everything on this 'onclick' map.closePopup(), layer.closePopup(), this.closePopup(), or invoking .getPopup() and then .closePopup() and still nothing - it doesn't work.
Of course I've tried to set the close button ({closeButton:true} as a param for popup) and it doesn't show.
Thanks in advance

there's no default closing button

That shouldn't be the case. Can you link to an example? (lots of people here use jsfiddle to demonstrate bugs)

I've tried almost everything on this 'onclick'

onclick is in disuse. Consider using querySelector and addEventListener instead.
onclick takes a function reference as a parameter, not a function call. Do read https://developer.mozilla.org/en-US/docs/Web/API/GlobalEventHandlers/onclick
onclick runs on the global scope. If your function is not defined in the global scope (i.e. as window.functionName), it won't work. Once again, use querySelector and addEventListener because they will run in the local variable scope.

@IvanSanchez better not to waste time making detailed assumptions before you see a JSFiddle test case :)

Hi, I've already tried it. On separated example it performs well and button shows, the popup is closing and so on.
Maybe it's because of angular-leaflet-directives using.
Will try @IvanSanchez advices.

Will reopen if reproduced in a minimal JSFiddle test case.

You can include a close button like this
newMarker.bindPopup(note, {closeOnClick: false, autoClose: false, closeButton: true}).openPopup();


Support for complex serialization

Attribute hydration has methods for both simple conversion - deserializeUsing() - and complex hydration - fillUsing().
In comparison, attribute serialization only has support for simple conversion - serializeUsing().
It'd be nice to have support for complex serialization (which would receive the model that's being serialized in the closure too) while defining the fields in the schema. That way we don't need to add extra Eloquent accessors on our models or create a dedicated resource class if we have just one custom attribute field.

Actually I did get this in the latest release - 1.0.0-beta.5.
Use the extractUsing() method, as documented here:

Thank you.


Show next (unread) message after delete doesn't work

Expected behavior
Show next (unread) message.
After deleting the actual shown message, K9 should go to either next unread message or next message in list.
Actual behavior
K9 shows next read message.
Instead of the expected behaviour, K9 shows the first read message in list and ignores all unread messages which are on higher positions.
Steps to reproduce

check setting "global settings - interaction - Show next message after delete"
uncheck setting "global settings - interaction - return to list after delete"
get some new unread messages in your inbox and also some read ones
go to unified inbox
read the first one and delete it
K9 will show the first read message instead of next in list (or next unread)

Environment
K-9 Mail version: 5.403
Android version: 7.1.1
Phone: Xiaomi Mi 6
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

I am experiencing exactly this issue. Quite annoying and reduces the usability of K9 for me. It seemed to start this behaviour after a recent update (perhaps in the last 8 weeks).
Thanks for a great app though!
K9 5.403
Samsung Galaxy S7
Android 7.0
POP3

Good to know that there are other people with other phones with the same issue, so I suggest this bug is present on all android phones and not limited to one special device. I don't know how long this bug persists on my phone, but I suggest it could be some time longer than you said, I suggest it is since the last update on 06.01.18. As a workaround I checked "global settings - interaction - return to list after delete", so it takes me back to list after deleting instead of showing me a wrong message, that saves one additional press on the back button, but is not a fix at all. I hope this bug will be fixed soon.

This appears to be the case if the sort order is "Read/unread". When the order is "Arrival", it seems to work as intended.

Fixed by #6271


Week 5 Autonomous

Week 5 goal: Score and drive off tarmac
This will need, at minimum, "drive distance" and "turn angle" commands, as well as automatic shooting.
Depends on #31

[ ] Create A Two Ball Path
[ ] Create Another Two Ball Path
[ ] Create A Three Ball Path
[ ] Create Another Three Ball Path

Done as of Week 5


Add networkId option to README options

This seems to be duplicating #256 Can you discuss with @longvinovleon the final wording?

Added in a different PR. Thanks!


feat(openshift): custom extra labels/annotations on deployments/pods

Welcome to Cryostat! 👋
Before contributing, make sure you have:

[x] Read the contributing guidelines
[x] Linked a relevant issue which this PR resolves
[x] Linked any other relevant issues, PR's, or documentation, if any
[x] Resolved all conflicts, if any
[x] Rebased your branch PR on top of the latest upstream main branch
[x] Attached at least one of the following labels to the PR: [chore, ci, docs, feat, fix, test]
[x] Signed all commits: git commit -S -m "YOUR_COMMIT_MESSAGE"

Fixes: #630
Description of the change:
This change adds allows the users to provide additional labels and annotations to be used in the cryostat deployment and pod.
Motivation for the change:
This change is helpful because some environment have a strong labeling policy for all resources.

/build_test

https://github.com/cryostatio/cryostat-operator/actions/runs/6263310857

Ah right, that CI command is currently not working: https://github.com/cryostatio/cryostat-operator/pull/619#issuecomment-1728475354

@OpenGuidou looks good! Just noticed one more thing. Could you run make bundle to regenerate the CSV to include these new properties? We recently upgraded to Operator SDK 1.31.0, so if you have another version installed locally you'll need to upgrade.

Ha I missed that, it's now done

@OpenGuidou Sorry one more thing. We require that all commits be signed in order for me to merge this PR:
https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

@OpenGuidou Sorry one more thing. We require that all commits be signed in order for me to merge this PR: https://docs.github.com/en/authentication/managing-commit-signature-verification/about-commit-signature-verification

Done !


✨ [Feature request]: use cdn links instead of images

Is your feature request related to a problem? Please describe.
Currently, we store images in our repository and would like to reduce the project size by utilizing CDN links.
Describe the solution you'd like.
currently, we are storing images in our repo, we need to use CDN links to minimize the project size
Describe alternatives you've considered.
.
Screenshot / Screenshare
No response
Additional context
No response
Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi.
Not a good idea in my opinion if I can voice it.
I'm on a very slow network, and other people too if I remember correctly some problems we had in May 2023.
The application itself is really slow. I've noted about 130 request on a page for the leaderboard when i was creating test for the page. It's a lot,. The application is not optimized enough.
It will be ok for the reactplay.io as it will not host the images, but will slow down the development in localhost for others.
Doesn't think
we can get an overall benefice from this.
I've send some modifications related to optimization of the script but as a whole project, it's not easy to move the boat so we can just take it with small steps.
Thinking about it, you can clone the plays repo on demand and ship only some basic in the main application.
And let the developper to download all the plays or only use the minumum provided.
When you want to create a play, you don't need to have 100 in your folder.

ok


Fix typo

Docs Build status updates of commit 486265c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint-server/xml/Microsoft.SharePoint.Taxonomy/ImportManager.xml
: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.


Search with insensitive case ?

+1 for me, would like to have an option to search with insensitive case.


CollectionView crash with EXC_BAD_ACCESS in RubyMotion

Using the pod in a rubymotion project just crashes on controller launch, tested using collection view.
Here's a test dummy app, just incase:
https://github.com/chrise86/DZNEmptyDataSet-test
Crashlog doesn't give much:
Process:               empty-test [59051]
Path:                  /Users/USER/Library/Developer/CoreSimulator/Devices/9AEF6684-00DC-4378-8434-8DAB36AC1571/data/Containers/Bundle/Application/8D3F8687-5523-437E-BEA3-FC58A74403E7/empty-test.app/empty-test
Identifier:            empty-test
Version:               1.0 (1.0)
Code Type:             X86-64 (Native)
Parent Process:        launchd_sim [86291]
Responsible:           launchd_sim [86291]
User ID:               501

Date/Time:             2014-12-23 07:19:32.223 +0000
OS Version:            Mac OS X 10.10.1 (14B25)
Report Version:        11
Anonymous UUID:        31D16FD2-D8DA-BFA3-450F-A5A584F8C004

Sleep/Wake UUID:       D555C7BA-5C28-4F90-8BE9-5DF0291CA26E

Time Awake Since Boot: 150000 seconds
Time Since Wake:       4000 seconds

Crashed Thread:        0  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       KERN_PROTECTION_FAILURE at 0x00007fff5f3ff638

VM Regions Near 0x7fff5f3ff638:
    MALLOC_TINY<PHONE_NUMBER>d00000-0000000111f00000 [ 2048K] rw-/rwx SM=PRV
--> STACK GUARD            00007fff5bc00000-00007fff5f400000 [ 56.0M] ---/rwx SM=NUL  stack guard for thread 0
    Stack                  00007fff5f400000-00007fff5fc00000 [ 8192K] rw-/rwx SM=COW  thread 0

Application Specific Information:
CoreSimulator 110.4 - Device: iPhone 6 - Runtime: iOS 8.1 (12B401) - DeviceType: iPhone 6

same kind of crash on swift

*** -[UICollectionView deviceDidChangeOrientation:]: message sent to deallocated instance 0x138874e00

@yasirmturk can this help?
https://github.com/dzenbot/DZNEmptyDataSet/pull/68

i am using swift let me try deinit

yes seems to be working


examples.PrintExporter and examples.LogExporter work differently

What version of OpenCensus are you using?
HEAD of this repo
What version of Go are you using?
go version go1.13.4 darwin/amd64
Details
examples.PrintExporter seems to require being passed to view.RegisterExporter, but examples.LogExporter can't be passed there because it doesn't implement the write Exporter type.  Instead, you're expected to run the Start method on it.
The documentation doesn't explain this difference and I'm not sure they were expected to be different.

@jmhodges LogExporter is using ExportMetrics interface instead of ExportView.
It creates IntervalReader on calling Start(). IntervalReader periodically calls ReadAndExport.
While it is not a bug, documentation does require an update.


Update mr-dev-days.md

Docs Build status updates of commit 83eac12:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/mr-dev-days.md
:white_check_mark:Succeeded
View

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.


Refs #27452 -- Added Default expression for use by serial fields.

https://code.djangoproject.com/ticket/27452

Just for reference, this depends on #11783
@InvalidInterrupt, I'd love your help in for of a review on that one too. Thanks for your support on the whole topic, it's much appreciated!

@codingjoe Could you point out what part of #11783 is necessary for this; I can't quite figure that out. It seems to me all planned uses of this API already function as I'd expect, but if there's something I'm missing I want to add a test case.

@InvalidInterrupt sorry, my statement was a bit misleading. It's not dependent, where you can't actually use the expression anywhere unless #11783 is implemented. If you consider this to be just an internal refactoring, your patch should be sufficient.
However, I wanted this be be used as a default field value not only for the AutoField but other fields too.
As is the patch doesn't seem to provide any new features.
Here's my suggestions, we wait for the other one, rebase and add the documentation again as well as a test that implements the default in another field than the AutoField. I'll do my best, to finish the other patch asap.
Sounds good?
Best, Joe

@codingjoe If I'm not mistaken, there's currently no way to provide a default for use by the database engine (it's sometimes used temporarily for things like adding a field tough); this is what would be needed to make the Default expression generally useful to users I think. Your PR seems to be focused on allowing expression defaults to work in DDL operations, and my experience is that I needed to explicitly avoid passing through the Default expression to the database in those cases, because ALTER TABLE foo ADD COLUMN bar ... DEFAULT DEFAULT; makes no sense.

I am not sure I can completely follow you. Would you mind expanding a bit more on what you are trying to convey?

@codingjoe It seems like what you're hoping for is that eventually a user could define a field like expiry = DateTimeField(default=Default(), db_default=Now()+timedelta(hours=2)); please correct me if this is not the case. To my knowledge Django currently does not have anything for users' field definitions like that hypothetical db_default parameter, and your PR's docs claim that you have not implemented it there. I think something providing the functionality of that db_default is what we would need before Django users would likely have much use for the Default expression.
Your PR allows Django to accept expressions as DB default used temporarily when adding or altering a field; the Default expression can't be used like that anyway because the DEFAULT keyword isn't a valid value for the column default in the database.
(If this still doesn't make sense please ask again or ask more questions; I feel like I'm having trouble explaining it concisely)

Wait, actually for me to do that, I would be great if you could open a new ticket. cc me, so I can accept it and move things ahead.

IMO, it's no longer needed since 7414704e88d73dafbcfbb85f9bc54cb6111439d3 introduced the DatabaseDefault expression.


Unable to make work a simple plugin

First of all, the documentation is very messy and seems like there are multiple way to do the same thing and no one knows which one is which.
For example what is the difference between this:
pluginserver_names = my-plugin,other-one

pluginserver_my_plugin_socket = /usr/local/kong/my-plugin.socket
pluginserver_my_plugin_start_cmd = /usr/local/bin/my-plugin
pluginserver_my_plugin_query_cmd = /usr/local/bin/my-plugin -dump

and this?
pluginserver_names = go,python

pluginserver_go_socket = /usr/local/kong/go_pluginserver.sock
pluginserver_go_start_cmd = go-pluginserver -kong-prefix /usr/local/kong/ -plugins-directory /usr/local/kong/go-plugins
pluginserver_go_query_cmd = go-pluginserver -dump-all-plugins -plugins-directory /usr/local/kong/go-plugins

pluginserver_python_socket = /usr/local/kong/python_pluginserver.sock
pluginserver_python_start_cmd = kong-python-pluginserver
pluginserver_python_query_cmd = kong-python-pluginserver --dump-all-plugins

For example, you could set Go and Python plugins like this (assuming an hypothetical Python plugin server called pypluginserver.py) but then there are different names?

plugins = bundled, go-hello, js-hello, py-hello where are those are coming from??.

pluginserver_<NAME>_query_cmd | Command to dump available plugins’ info | /usr/local/bin/query_<NAME> So it has a _query prefix or not?

This is my docker-compose
  apiGateway:
    image: kong/kong-gateway:<IP_ADDRESS>-alpine
    volumes:
      - ./kong.yaml:/usr/local/kong/declarative/kong.yaml
      - ./ratelimiter:/usr/local/kong/ratelimiter
    environment:
      - KONG_DATABASE=off
      - KONG_DECLARATIVE_CONFIG=/usr/local/kong/declarative/kong.yaml
      - KONG_ADMIN_LISTEN=<IP_ADDRESS>:8001
      - KONG_PLUGINS=bundled,ratelimiter
      - KONG_PLUGINSERVER_NAMES=ratelimiter
      - KONG_PLUGINSERVER_RATELIMITER_SOCKET=/usr/local/kong/ratelimiter.socket
      - KONG_PLUGINSERVER_RATELIMITER_START_CMD=/usr/local/bin/ratelimiter
      - KONG_PLUGINSERVER_RATELIMITER_QUERY_CMD=/usr/local/bin/ratelimiter -dump

this is my plugin:
package main

import (
	"github.com/Kong/go-pdk"
	"github.com/Kong/go-pdk/server"
	"log"
)

type Config struct {}

func New() interface{} {
	return &Config{}
}

func (c Config) Access(kong *pdk.PDK) {
	log.Println("hello custom plugin")
}

func main() {
	err := server.StartServer(New, "1.1", 1000)
	if err != nil {
		log.Fatal(err)
	}
}

and I get this error:
apiGateway_1  | 2021/07/14 06:37:53 [notice] 1#0: [lua] license_helpers.lua:123: read_license_info(): [license-helpers] could not decode license JSON: No license found
apiGateway_1  | 2021/07/14 06:37:53 [notice] 1#0: [lua] license_helpers.lua:123: read_license_info(): [license-helpers] could not decode license JSON: No license found
apiGateway_1  | sh: /usr/local/kong/ratelimiter: not found
apiGateway_1  | 2021/07/14 06:37:54 [error] 1#0: init_by_lua error: ...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:166: Not a plugin info table:
apiGateway_1  | /usr/local/kong/ratelimiter -dump
apiGateway_1  |
apiGateway_1  | stack traceback:
apiGateway_1  | 	[C]: in function 'error'
apiGateway_1  | 	...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:166: in function 'ask_info'
apiGateway_1  | 	...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:202: in function 'get_plugin_info'
apiGateway_1  | 	...local/share/lua/5.1/kong/runloop/plugin_servers/init.lua:269: in function 'get_plugin'
apiGateway_1  | 	...local/share/lua/5.1/kong/runloop/plugin_servers/init.lua:277: in function 'load_plugin'
apiGateway_1  | 	/usr/local/share/lua/5.1/kong/db/dao/plugins.lua:146: in function 'load_plugin_handler'
apiGateway_1  | 	/usr/local/share/lua/5.1/kong/db/dao/plugins.lua:234: in function 'load_plugin'
apiGateway_1  | 	/usr/local/share/lua/5.1/kong/db/dao/plugins.lua:286: in function 'load_plugin_schemas'
apiGateway_1  | 	/usr/local/share/lua/5.1/kong/init.lua:578: in function 'init'
apiGateway_1  | 	init_by_lua:3: in main chunk
apiGateway_1  | nginx: [error] init_by_lua error: ...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:166: Not a plugin info table:
apiGateway_1  | /usr/local/kong/ratelimiter -dump
apiGateway_1  |
apiGateway_1  | stack traceback:
apiGateway_1  | 	[C]: in function 'error'
apiGateway_1  | 	...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:166: in function 'ask_info'
apiGateway_1  | 	...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:202: in function 'get_plugin_info'
apiGateway_1  | 	...local/share/lua/5.1/kong/runloop/plugin_servers/init.lua:269: in function 'get_plugin'
apiGateway_1  | 	...local/share/lua/5.1/kong/runloop/plugin_servers/init.lua:277: in function 'load_plugin'
apiGateway_1  | 	/usr/local/share/lua/5.1/kong/db/dao/plugins.lua:146: in function 'load_plugin_handler'
apiGateway_1  | 	/usr/local/share/lua/5.1/kong/db/dao/plugins.lua:234: in function 'load_plugin'
apiGateway_1  | 	/usr/local/share/lua/5.1/kong/db/dao/plugins.lua:286: in function 'load_plugin_schemas'
apiGateway_1  | 	/usr/local/share/lua/5.1/kong/init.lua:578: in function 'init'
apiGateway_1  | 	init_by_lua:3: in main chunk

Hi @MatteoInfi, there're no technical difference between the two config snippets you posted at first,
but they are to describe different use cases. The first one is example for "embedded server", where
each plugin is a standalone microservice; the second is for sharing multiple pugins in same plugin server.
In docs they below to different sections and should be read based on their context.
plugins = bundled, go-hello, js-hello, py-hello those are example names of plugins. As described in the docs,
those are simply examples.
The query_ prefix in pluginserver_python_query_cmd is the default value, this is correct.
Back to the error you are facing, sh: /usr/local/kong/ratelimiter: not found suggested that the plugin binary
is not found. It looks a configuration error to me.

Hello, any help on this bug?

@MatteoInfi please see my reply above

@fffonion Yes, I saw it, however it does not solve my problem, I followed the tutorial on the Kong docs, but is not working

@MatteoInfi The error looks like a environment-specific issue to me. Maybe try to exec into the container and double check
if /usr/local/kong/ratelimiter is indeed there and accessible.

@fffonion I have already checked that the binary is in that folder and is runnable

is the plugin in /usr/local/kong/, or /usr/local/bin/?

I have tried both.

of course, you have to choose one and be consistent.
In your compose file, i see you're volume-mapping a file to /usr/local/kong/ratelimiter, i guess this is the plugin executable.  personally, i don't trust volume-mounting single files, as it might make privilege flags more confusing than usual.  Still, i guess this could work.
in the same compose snippet, i see that the commands to run expect /usr/local/bin/ratelimiter, just be sure to have them agree with the volume mapping.
finally, you're using the Alpine Linux version of kong.  that means the Go plugin must be compiled for a musl-based system.
The error messages in your log file say that running /usr/local/kong/ratelimiter -dump doesn't result in a JSON description of the plugin.  The easiest way to diagnose it is to manually run the exact same command from within the container.  that way you'd find which of several possible issues you have:

command not found
no executable permissions
can't link with musl library
bad JSON output

@javierguerragiraldez

i don't trust volume-mounting single files

I don't either, but it is just for quick testing.
Anyway, thanks for your tips, I will try build and run the binary directly inside the container.

Finally solved, the configuration was correct, however this was the problem:

you're using the Alpine Linux version of kong. that means the Go plugin must be compiled for a musl-based system

Many thanks for the tips.


Add process mutex API / option to prevent multiple instances of the game

Is there a way to prevent the game from being launched multiple times? I don't know if this behavior is similar on other platforms but on Windows, I can keep pressing Enter on the *.exe file and it will just keep opening. This might be useful for some people but I would like the option to make it so if the game was launched a 2nd or 3rd time, it will just refocus on the already running instance of the game.

I think a option to just check if an game instance is running already to send a OS specific message alert stating the game is already running. Simple, quick and easy for cross-platform. (i think mobile already handles this automatically)

Nothing built-in to Godot, as far as I know. Needing this too for a current project.
Some keywords for this: interprocess (or cross-process) synchronization, locks/mutexes/semaphores.
There are POSIX and Windows APIs to do this (something like this).
Would be a nice feature. Until Godot offers an cross-platform abstraction for this, you have the following options:

Create a file upon launch, delete upon exit. At start, check if it already exists, if so, exit with error. Note that this is probably the easiest, but can be error-prone on crashes if you don't make sure your lock file is always deleted
Check list of running processes at start, see if your executable is already running (using OS.shell_open()). Note that this is platform-dependant, as you need to call something like ps on Unix and tasklist on Windows. Also error-prone, e.g. tasklist may require permissions or is not available on certain installations. May also be detected by AV heuristics, as both shell usage and listing processes is at least suspicious.
If you're using C# in Godot, or are willing to, you can use a third-party C# library that implements this, like the one I linked above
You could use a third-party library or the Windows/Unix API calls directly to do this to implement a C++ module for Godot
You can use networking; simply create an PacketPeerUDP and listen on a port. If this fails because that specific port is already occupied, it is likely that your game is already running in another process.

Currently I'm doing 5, as its easy and automatically frees the lock (port) on exit/crash.

In some cases it could be intended to run multiple instances of one game, e. g. testing multiplayer.

In some cases it could be intended to run multiple instances of one game, e. g. testing multiplayer.

Agreed, it should be optional

You can use networking; simply create an PacketPeerUDP and listen on a port. If this fails because that specific port is already occupied, it is likely that your game is already running in another process. Note that this can trigger firewall warnings, so if you're not doing multiplayer, may look weird.

If someone its searching for an implementation for this, here its mine:
# Remember to AutoLoad as first Singleton!
extends Node

# "Port + i" where to check if instance number is already running or not
# be sure that BASE_PORT + [ range of desired instances ] are NOT allocated
# by other services
const BASE_PORT = 2000
const LISTEN_IP = "<IP_ADDRESS>"

# This is the variable that will hold the instance number for this game
var GameInstanceNum: int setget noset

func noset(_a):
	assert(false)

func _init():
	var sv = TCP_Server.new()
	var port_offset = -1
	while true:
		port_offset += 1
		var port = int(BASE_PORT + port_offset)
		if sv.listen(port, LISTEN_IP) == OK:
			break
	GameInstanceNum = port_offset

I found that my implementation don't work correctly in Windows. <IP_ADDRESS> don't exist in windows but with <IP_ADDRESS> also doesn't work. I created a gist with two implementations, a port based option for Linux and a file based option for Windows.
In the file based check, it works in Windows by creating a file and keep it opened, so when another instance tries to open that file, it can't because its locked. This way if the game crashes it also releases the look of that instance ID, this implementation somehow only works in Windows and not Linux. So because I just need this to work inside the editor for quick launching two instances, one as a server and all the others as clients, this fix works for me, but I don't recomend to use this implementation for production.
Gist

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Auth-guard rad clone command

Also have a look if other commands we already have should get the same treatment.

Changes:
🩹 Fixes

auth: address multiple UX issues with the auth request flow

use warning kind of notification instead of error when attempting to perform an auth-guarded command, since the user is also offered the option to launch the auth flow
improve copy of notification requesting auth for guarded commands
add missing log entry when the user dismisses the authentication request

commands: address multiple UX issues with rad clone

require authentication before running command (align with CLI's behaviour)
don't explain erroneously that RID means Radicle ID
replace the term "project" with "repo" in user notifications
remove duplicate word on successful clone notification


question - directories colors

is it possible to change the colors of directories? if not, how can I contribute to its existence?

We have to add white color for folder but by changing their names.
then we will have original folders and white folder (android_folder_w.svg) for white

thank you!


Add a check for EnforcedStyle: (default) examples

Previously, not all default enforced styles were at the top.
Known limitations:

only checks for EnforcedStyle:, not e.g.  EnforcedStyleForMultiline: (which is the only) enforced style type in Style/TrailingCommaInArrayLiteral cop.

only checks for EnforcedStyle: when other enforced style types are present, e.g. EnforcedStyleForEmptyBrackets: for Layout/SpaceInsideArrayLiteralBrackets (also has regular EnforcedStyle:).

No autocorrection 😄

Follow-up to https://github.com/rubocop/rubocop-rspec/pull/1289, basing on this discussion.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[-] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[-] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[-] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Is it worth adding a changelog entry? It has user-facing changes if RuboCop is used as a library to build cops in the user codebase, and if CopsDocumentationGenerator is used (which seems very unlikely).

cc @koic

It's okay as it is - I view this mostly as a documentation change. Thanks!


Added Local Authority search to Find Organisation page

Context
AB#207649 AB#206189
Change proposed in this pull request
Added the ability to search by Local Authority on the Find Organisation page.
Guidance to review
LA search should function in a similar way to the School and Trust search. The suggestions on matching name or code should appear for the user to select from. Validation errors should be displayed in the same way. Once the form is successfully submitted, it is currently intentional that the home page is redirected to in lieu of the Local Authority landing page (see the todo).
Checklist

[x] Work items have been linked (use AB#)
[x] Your code builds clean without any errors or warnings
[x] You have run all unit/integration tests and they pass
[x] Your branch has been rebased onto main
[x] You have tested by running locally

Web (Build artifacts Format solution) Failing after 53s — Build artifacts Format solution failed

I literally formatted FindOrganisationController.cs in Rider just before pushing. Helpfully, its rules seem to differ from dotnet format. Fixing now


chore: bdd-nh-1585688683 to 0.0.1

chore: Promote bdd-nh-1585688683 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Enable/Disable scrollbar

Thanks for the nice scrollbar plugin. Is it possible enable/disable scrollbar from the controller, similar to jquery function -> http://manos.malihu.gr/repository/custom-scrollbar/demo/examples/disable_destroy_example.html

Hi @gayanvirajith, if you're using version 0.0.10 or later then you should be able to do something like described here: https://github.com/iominh/ng-scrollbars/tree/0.0.10#updating-scrollbars
To enable/disable, you would probably call
$scope.updateScrollbar('update'); // enable
$scope.updateScrollbar('disable') // disable

Maybe, add a parameter to the directive like '...Disabled' : false could be a better option (juste return to avoid next steps and build of the scroll) instead of build the scrollbar and then destroy it.
Anyway, I'm here because I do not understand how to achieve this.
Always got an error like 'this is not a function'.
Any help please ? Thanks.


FEF: Cannot edit feature attributes

repro steps:
create FEF app that consumes service with all editing capabilities
Open filter/edit widget
click feature in map - editing popup does not appear

Caused by bug in JSAPI - resolved


v1.8.0

I'd like to release 1.8.x rather than 1.7.x since we added the new faraday and net-http-persientent dependencies. thoughts?

:+1:


fix(artifacts): unmarshaling error

What type of PR is this?

Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

PR #1637 supersedes this.


Add a Ticker struct to memoryless.

This allows package users to use it just like a time.Ticker, which makes everyone's life easier.

This change is

Pull Request Test Coverage Report for Build 543

48 of 49 (97.96%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 80.895%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

memoryless/memoryless.go
48
49
97.96%

Totals

Change from base Build 535:
0.5%

Covered Lines:
741

Relevant Lines:
916

💛  - Coveralls


Rewrite size expressions of constant-sized constexpr arrays

Enhances the support for rewriting the size expressions of constant-sized arrays so that arrays with the constexpr qualifier are also handled.
Fixes #283.

Merged!


fix: renaming staging branch to dev

What would you like to be added?
Suggestion to rename staging to dev branch to align with other open resource repo.
Anything else you would like to add?
No response
Are you willing to submit PRs to contribute to this feature?

[ ] Yes, I am willing to implement it.

This would involve:

updating branch settings to set dev branch as the default branch
update release note to capture details around rebasing between dev and main
updating workflow to run against dev branch


Is functionless still alive

Dear maintainers of functionless ,
I hope this message finds you well. I was wondering if the functionless package is still being supported. It's been several months since the last commit to the repository, and I wanted to check if the package is still being maintained.
Thank you for your time and any information you can provide.

Hi @MaazDev, thanks for reaching out and for your interest in functionless. We're not actively working on the project at this time. Given feedback, we decided to double down on a more narrow approach that is not coupled to AWS or the AWS CDK. It is similar in theme and industry. If you'd like, I'd be happy to meet with you to provide a demo. We're just about to start running a beta of the new project so if that's something that interests you, let's connect.

Thanks for getting back to me and for letting me know what's going on with functionless. I appreciate the offer to meet and see a demo of the new project, but I'm just working on personal projects right now and don't have a company to use it for. No worries though! I hope the new project goes well and wish you all the best. Thanks again for taking the time to respond.

Hello, I've also been eyeing functionless for a while - sad to see it be deprecated, but somewhat interested in what you are working on now. Is it an open source project, similar to functionless in design goals?
We have many step functions and the stringly typed system that functionless solved is what we are looking for.

Hi @robert-lemmens, I am working on an open source project that has similar goals but a slightly different approach. If you found Functionless interesting, you might be interested in our new project. Let me know if you'd like to chat about it!

hi @sam-goodwin, I might be, it all depends.  Is there anywhere i can find some information on your new project?


failing tests with OpenRefine 3.8 due to change in escaping of \

failing tests with OpenRefine 3.8.7:
[2024-12-08T22:27:47] FAILED export-tsv.sh with exit code 1!
[2024-12-08T22:27:48] FAILED import-csv-columnNames.sh with exit code 1!
[2024-12-08T22:27:48] FAILED import-csv-encoding.sh with exit code 1!
[2024-12-08T22:27:48] FAILED import-csv-guessCellValueTypes.sh with exit code 1!
[2024-12-08T22:27:48] FAILED import-csv-headerLines.sh with exit code 1!
[2024-12-08T22:27:48] FAILED import-csv-ignoreLines.sh with exit code 1!
[2024-12-08T22:27:48] FAILED import-csv-ignoreQuoteCharacter.sh with exit code 1!
[2024-12-08T22:27:49] FAILED import-csv-includeArchiveFileName.sh with exit code 1!
[2024-12-08T22:27:49] FAILED import-csv-includeFileSources.sh with exit code 1!
[2024-12-08T22:27:49] PASSED import-csv-limit.sh
[2024-12-08T22:27:49] PASSED import-csv-projectTags.sh
[2024-12-08T22:27:49] PASSED import-csv-quiet.sh
[2024-12-08T22:27:49] FAILED import-csv-quoteCharacter.sh with exit code 1!
[2024-12-08T22:27:49] FAILED import-csv-separator.sh with exit code 1!
[2024-12-08T22:27:50] FAILED import-csv-skipBlankRows.sh with exit code 1!
[2024-12-08T22:27:50] FAILED import-csv-skipDataLines.sh with exit code 1!
[2024-12-08T22:27:50] FAILED import-csv-trimStrings.sh with exit code 1!
[2024-12-08T22:27:50] PASSED import-csv-unicode.sh
[2024-12-08T22:27:50] FAILED import-csv.sh with exit code 1!
[2024-12-08T22:27:50] FAILED import-json-recordPath.sh with exit code 1!
[2024-12-08T22:27:51] FAILED import-json-rename.sh with exit code 1!
[2024-12-08T22:27:51] FAILED import-json-storeEmptyStrings.sh with exit code 1!
[2024-12-08T22:27:51] FAILED import-json-trimStrings.sh with exit code 1!
[2024-12-08T22:27:51] FAILED import-json.sh with exit code 1!
[2024-12-08T22:27:51] FAILED import-jsonl.sh with exit code 1!
[2024-12-08T22:27:51] FAILED import-tsv.sh with exit code 1!

seems to be related to a change in escaping of \
--- export-tsv.assert	2024-12-08 22:32:59.513242150 +0000
+++ export-tsv.output	2024-12-08 22:32:59.653242145 +0000
@@ -1,4 +1,4 @@
 a	b	c
 1	2	3
 0	0	0
-$	\	'
+$	\\	'

--- import-csv-columnNames.assert	2024-12-08 22:33:00.089242130 +0000
+++ import-csv-columnNames.output	2024-12-08 22:33:00.237242125 +0000
@@ -2,4 +2,4 @@
 a	b	c
 1	2	3
 0	0	0
-$	\	'
+$	\\	'

fixed with https://github.com/opencultureconsulting/orcli/commit/c069605ae00bfb6dbc8ab962edea914855f1e0a9


Removed curl cert validation disable

I'm guessing this was done because some PHP installations don't have properly configured root certificates for cURL. This is something that I think the server admin should fix in the PHP config instead of it being "fixed" by making the request vulnerable.
Any decent hosting company should be providing the root CA bundle and PHP on Linux will have it available in most standard configurations. On Windows the bundle does need to be manually configured but it is really easy to do.

I vote for a config entry to toggle this

Certificates validation should really be optional (and turned off by default): I've seen many server installations that don't have root certificates setup correctly.
Furthermore concrete5 can be installed with hosting services, where there's very limited control on the server setup.

Really feel it should be on by default and optional.
I think it says something that false isn't curl's default behavior, to protect people who don't know any better. If you add it as an option and set it as false nobody is going to even bother using it except the few people that a) know the setting exists and b) actually care about it.
For instance, back when heartbleed happened many certificates were revoked because the certificates were actually compromised or potentially compromised. At that point I really don't want my website to be trying to authenticate to something with an invalid / insecure certificate and giving away my or my users credentials.

Im wondering if we could add 2 setup checks, 1 for internet connectivity, and 1 for a ssl check, it would look like so:
If Internet passes but ssl fails show a warning that some stuff won't work
If Internet fails then show a warning that we were unable to check for ssl (along with the warning there is no connectivity)
it could be checked by making 2 curl requests to a known good url supporting both http and https (eg google.com) the internet connectivity check is more for sanity than anything else.
regardless, there needs to be a config option to enable/disable

definitely not disable it by default. Either check them or show a warning, everything is dangerous. I agree that there needs to be an option, there are so many bad PHP hosts, it's a shame though, at the end they will blame the product for doing unsafe stuff but I guess it's the way it has to be..

I've finally updated this based on the previous discussion. I added a check to the installer that tests is the ca bundle is available. If it is not, peer validation is disabled and the user is shown a warning. Thoughts?

I think this should require direct deliberate action to be disabled by the user, never verifying peer on these could be a pretty serious issue with a targeted MITM attack.

Another thought, do we currently require an internet connection to install concrete5?

Here's my suggestion:

Don't test this on install at all.
Don't disable peer validation by default.
Fix curl calls so that they properly handle the possible curl errors or make a curl wrapper that will do that for us.
Make a dashboard page that lets you disable peer validation.

@KorvinSzanto I like that option.

Do we already have a cURL wrapper? It looks like the options in composer are a plenty.

Does FlySystem have something?

We don't have a curl wrapper. I don't believe flysystem has anything.
On Mon, Mar 30, 2015 at 3:18 PM, Michael Krasnow<EMAIL_ADDRESS>wrote:

Does FlySystem have something?
—
Reply to this email directly or view it on GitHub
https://github.com/concrete5/concrete5-5.7.0/pull/1870#issuecomment-87856421
.

I'm thinking of two different ways to implement this.

Have a simple factory that returns a curl handle with proxy and ssl options pre-configured
Have a full wrapper for curl

@KorvinSzanto

@EC-Chris These are not distinct solutions, either way we are going to need a factory class that is swappable, the question here is whether it returns a resource like curl_init does or an instance of some curl wrapper.
A few benefits that we can get from a wrapper instance:

Mock in tests
Count the total sent http requests in a page load
Count the total curl requests left open at the end of the page load
Re-implement the curl interfaces to use something other than curl

A few detriments:

Some existing libraries are very heavy and feature rich
Some of the lighter libraries are feature poor and could prevent implementing a better wrapper later
Curl resources are used everywhere and switching to an instance is a huge undertaking

With those things said, I think the right thing to do here is to wrap an existing curl wrapper library.

Yeah, if we do this we want to do it right – seems like a "curl-specific" wrapper is kinda outdated anyway. What if we moved to this or something similar ?
http://phphttpclient.com
I have no experience with this library.

@aembler I would advocate https://github.com/guzzle/guzzle if we are planning to include a full http client, guzzle is a member project of the PHP-FIG and is super active,
The only problem is that guzzle is php 5.4+ and has been for quite some time. Maybe we should hold off on adding something like this until we can support 5.4? Centos 7 has 5.4 by default and the market share is getting close to surpassing 5.3 so we might be able to require 5.4 soon.

PS: Not that this should really be used as a metric, but guzzle has more downloads today than httpful has in the last 30 days so that might count for something.

guzzle is great, never had any problems nor wishes for things it doesn't support!

I'd prefer guzzle over httpful, but if dropping support for 5.3 is far down the road it would probably be best to go for something like httpful.
If its decided to wait and implement guzzle when support for 5.3 is dropped can we at the very least enable verify peer by default for curl calls in the core with a config setting to disable (which is what my PR is at now)?

Centos 7 has 5.4 by default and the market share is getting close to surpassing 5.3 so we might be able to require 5.4 soon.

@KorvinSzanto Well, I don't think that we will be able to switch to 5.4 soon... From the data you linked:

5.3 is satisfied by the 81.2% of users (sum of all versions ≥ 5.3)
5.4 is satisfied by the 34.8% of users (sum of all versions ≥ 5.4)

You will never get a higher number for 5.4 if you sum like that,  by that
logic 5.2 is satisfiable by 98.8%....
The important question is how hard is it for the 10% difference to upgrade
beyond 5.4 if they must since a lot of these are set up and left websites,
and I contend that at this point it is not that hard. You don't have to
worry about BC,  and most shared hosts probably already offer php 5.4 by
default. It would be interesting to audit some shared hosts to see what
they support today.
On Wed, Apr 15, 2015, 2:10 AM Michele Locati<EMAIL_ADDRESS>wrote:

Centos 7 has 5.4 by default and the market share is getting close to
surpassing 5.3 so we might be able to require 5.4 soon.
@KorvinSzanto https://github.com/KorvinSzanto Well, I don't think that
we will be able to switch to 5.4 soon... From the data you linked
http://w3techs.com/technologies/details/pl-php/5/all:

5.3 is satisfied by the 81.2% of users (sum of all versions ≥ 5.3)
5.4 is satisfied by the 34.8% of users (sum of all versions ≥ 5.4)

—
Reply to this email directly or view it on GitHub
https://github.com/concrete5/concrete5-5.7.0/pull/1870#issuecomment-93274832
.

@EC-Joe Waiting for 5.4 and keeping what this PR supplies is what I'm advocating


Use scratch in docker_sample project

Using scratch will make the image smallest possible size to 8Mb.

thanks!


failed to cherry pick PR 3526 from ty-dc, to branch release-v0.7

commits 1d2e1c81093a33d92c2f0e5ea65f7cfdee714f41 of ty-dc conflict when merging to branch release-v0.7, please manually cherry pick it by yourself.
PR https://github.com/spidernet-io/spiderpool/pull/3526 ,  action https://github.com/spidernet-io/spiderpool/actions/runs/9541331074
Auto-merging .github/workflows/auto-upgrade-ci.yaml
Auto-merging test/Makefile
Auto-merging test/e2e/annotation/annotation_test.go
CONFLICT (content): Merge conflict in test/e2e/annotation/annotation_test.go
Auto-merging test/scripts/debugEnv.sh
CONFLICT (content): Merge conflict in test/scripts/debugEnv.sh
error: could not apply 1d2e1c81... Merge pull request #3526 from ty-dc/debug-upgrade-x
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
hint: Disable this message with "git config advice.mergeConflict false"

Only the latest three branches are maintained, and we skip release-v0.7.


DWN SDK should support delegated signing

Currently the DWN SDK doesn't support delegated signing. Web5 connect requires delegated signing.

stale


Analyse Linting Results

Ran lint on variant offRelease: 1179 issues found
Ran lint on variant obfDebug: 1178 issues found
Ran lint on variant opffRelease: 1184 issues found
Ran lint on variant obfRelease: 1178 issues found
Ran lint on variant offDebug: 1179 issues found
Ran lint on variant opffDebug: 1184 issues found
Wrote HTML report to file:///home/travis/build/openfoodfacts/openfoodfacts-androidapp/app/build/reports/lint-results.html
Wrote XML report to file:///home/travis/build/openfoodfacts/openfoodfacts-androidapp/app/build/reports/lint-results.xml```

I've fixed 2 lint errors

@teolemon What exactly needs to be done? Can you please explain in detail? If all these are to be corrected or checked, then what should be the protocol? (Directly submitting a PR or first opening a particular issue followed by a discussion and then a PR ). Please help.


feat: 🎸 #5: Add summary as an output

This pull request adds the summary of the action as an additional output.
closes #5
If the user wants, they can use this summary and do further processing, e.g. send an email with the outdated table, automatically comment on a pull request, etc.
I have added a simple example with echo to the README, but it could also show examples for sending emails or commenting on PRs, I'm not sure what's best.
- uses: gh640/npm-outdated-action@main
  id: npm-outdated
- name: Comment PR
  uses: thollander/actions-comment-pull-request@v3
  with:
    message: ${{ steps.npm_outdated.outputs.summary }}
    comment-tag: npm-outdated

Results in

@frankvollebregt Thank you for sharing the issue and the PR. Overall it looks good!
I'd like to confirm one thing. I feel like using core.summary.stringify() might be better than core.summary._buffer. What are your thoughts? If stringify() is better, I'd appreciate it if you could change it. (Please see the inline comment ↑ .)

If this (or another version) gets merged, can you then also publish/tag the new version so I can use the marketplace version?
Understood!

@frankvollebregt
It looks fine and I've merged it. Thank you :)

@frankvollebregt
I pushed a new tag v1.2.0 and published a new release.
https://github.com/gh640/npm-outdated-action/releases/tag/v1.2.0
Also, I forced v1 to point the latest commit. I believe we can use the new summary output with v1.
- uses: gh640/npm-outdated-action@v1

If you have any problems or issues with the new v1 or v1.2.0, please let me know!


Set Azure CNI default mode to l2bridge

What this PR does / why we need it:
This PR

sets default operating mode of Azure CNI to l2bridge on Windows nodes.
makes kube-dns the default DNS server on Windows nodes.
Special notes for your reviewer:
This PR requires Azure CNI PR#155

@sharmasushant @jackfrancis pls review. Thanks!

/lgtm pending E2E

/approve

@dineshgovindasamy FYI also includes dns order fix


[Feature Request] Automatic renaming device

This block would allow a computer to automatically rename items by putting items into its slot, running peripheral.rename(name:string) on it and then pulling the item from the block's inventory
suggested crafting recipe:

initial comment edited to fix some formatting issues

Thanks for the feature request. I can't think of any way to make this work within the paradigm of CC without introducing a whole new block. It's not trivial, or expected, for other peripheral mods to interact with CC's item transfer functions. And adding a whole new block for this just seems overkill.
The most CC-native approach that comes to mind is allowing turtles to use anvils in the same way as crafting tables, making use of a number of their internal slots, but you still have to consider how XP would be spent, and I don't think fuel is an appropriate equivalent.
With those points in mind; unless something really good comes along, I don't think I'm going to implement this.


Pages disappearing after a merge

I have not yet managed to isolate the problem in more detail, or do more research to see if people have experienced this problem - but it is an urgent issue for a customer so am posting it here. I'm not even sure if it's a bug or a missing feature.
Issue Summary
Loading the dumped data into a new Wagtail instance from two separate sites, which had completely separate page collections, results in some sub-pages mysteriously missing.
Steps to Reproduce
I have not yet managed to reproduce this on a fresh instance. If someone wants a copy of my customer's data for testing, please contac tme directly.
Technical details

Tested on Python 3.7 and Wagtail 2.9

@loleg To clarify, you loaded dumped data from two completely separate Wagtail instances into one? If this is the case, this is likely because some pages have the same ids, so overwrite each other: you could use a solution like Wagtail Transfer (https://github.com/wagtail/wagtail-transfer) to work around these types of issues

@jacobtoppm beat me to it :-) Looking at the Django docs, it seems that loaddata is only really intended for populating an empty database with initial data. I don't think it would be feasible to support merging multiple data sets that way, particularly as Wagtail's page tree adds an extra level of complication (the internal path and numchild fields need to be set up to account for parent/child relations, and importing these from elsewhere is liable to produce inconsistent data). I reckon wagtail-transfer is indeed the way to go here.

Thanks a lot for the advice!
To be precise, one server's database was an extension of the other, not completely separate / divergent trees, but thanks for the pointer, I agree that Page ID creep is the most likely problem. Strange that this is not being detected in the import, e.g. through UNIQUE constraints.
I still think there is an issue here, albeit a slightly different one that was originally posted: namely, that the current documentation makes no mention of Wagtail Transfer - which I'll need to dig into and learn more about - or dumpdata, for that matter, except in the section about fixtures, even though I find it a very versatile tool for maintenance & recovery.


Make googletest not be a git submodule, and instead clone it separately in the CI script

This PR is supposed to address this request: https://github.com/AngusJohnson/Clipper2/pull/737#issuecomment-1842430219

I think I would personally prefer the submodule approach (in fact I added it in the first place), but I could certainly live with this one, too.

Thanks!

I think I would personally prefer the submodule approach (in fact I added it in the first place), but I could certainly live with this one, too.

Thank you for the PR Juha. As you know I have an extremely limited understanding of CMake. Nevertheless, I'm confident that you're changes are fine. But are you suggesting there's a better way? I've followed your link and it seems to me that I committed your PR#44 back then, so I don't understand what you're suggesting would be better?

Sorry for being unclear, Angus! I think it's a matter of taste, and if this change helps @yoavmil, then I'm fine with this.


Fails after API change

I get a catastrophic failure of the system (blank screen) if I try to update it to commit ddce0412ac8fe675153182909d82955c79d1f660 or beyond. This looks like some kind of API change.
A corresponding entry in the error log reads:
2015/12/04 10:20:05 [warn] 3375#0: *36 an upstream response is buffered to a temporary file /var/lib/nginx/fastcgi/3/00/0000000003 while reading upstream, client: IP, server: apespace.org, request: "GET /zotfeed?f=&mindate=2015-12-02+06%3A02%3A08 HTTP/1.1", upstream: "fastcgi://unix:/var/run/php5-fpm.sock:", host: "apespace.org"

@bashrc  like that start also my server and become a big fail crash with the MYSQL . A very good advice backup your database now . The problems start with one changes pull from github and after I restore the server I believe all is working fine. Now I have all the users broken down and my own user lose

For anyone else concerned about this the last known good commit is 609e42cdd8facae0a673bb1a370c44225d22b1cd
If you checkout to that then you can avoid having a "big fail crash"

@bashrc I already report when I have the big mesh and they say is my server configuration after of waste hours checking all I got a very big concern when Y see MariaDb already make several bak files
I try to recover that but with not luck . Really all the best I hope some one can help you and not become in the mess I am now . I go sleep 5 AM for try to recover all my users some I think can export now the profiles other I have really dubts

I can nott confirm any issues with current git...

@git-marijus redmatrix say that to me see please some of my close tickets. Anyway there serious issues still are not solve MYSQL crash , High Memory in PHP with process Apache refuse to close setting all according to the Install documents . SettingUP according to My.cnf module prefork . I am not the only you have also kerenstivo this guy me. Possible many issues is we need better documentation no only about install also about configurations and solutions to possible errors. If you can provide that people will not go around making theory  like the song que sera sera what ever will will be

@bashrc A catastrophic failure is usually the easiest to diagnose in PHP. Look in /var/log/apache2/error.log or wherever your web server stores error logs. The relevant lines will be a stack trace showing exactly the line of the bug and the call trace leading up to it. Post it here and it can be tracked down properly, possibly duplicated, or fixed.

The error mentioned above was all I could find in the log.

what db backand are you using?

mariadb

@bashrc The problem here is the ambiguous use of the word "the". You are looking at "a" log not "the" log. The error above appears to come from the hubzilla log, the one enabled in the admin screens inside hubzilla. The error that would help you track down the crash is not going to ever appear in that log, because when it happens, Hubzilla itself has crashed and can't write to is own log.
 I got basically same setup and no problems...

I'm not sure what "Expires: Thu, 19 Nov 1981" is about.

I think it can possibly be that you have some addons installed that i have not and this commit breaks something in the addon.
I see one thing in include/api_auth.php which is probably wrong (from the commit you mentioned in your top level post)
in line 3
require_once("oauth.php");
should probably be
require_once("include/oauth.php");

actually though i see that befor the change it was also require_once("oauth.php");
well...

I'm not sure what "Expires: Thu, 19 Nov 1981" is about.

is your hw clock set right?

The date looks ok. The only plugin I had enabled was Diaspora and a NSFW addon. Disabling those doesn't make a difference.

Another log:
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL NPN advertised
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL_do_handshake: -1
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL_get_error: 2
2015/12/06 13:36:01 [debug] 19878#0: *3 reusable connection: 0
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL handshake handler: 0
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL_do_handshake: 1
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL: TLSv1.2, cipher: "ECDHE-RSA-AES128-GCM-SHA256 TLSv1.2 Kx=ECDH Au=RSA Enc=AESGCM(128) Mac=AEAD"
2015/12/06 13:36:01 [debug] 19878#0: *3 reusable connection: 1
2015/12/06 13:36:01 [debug] 19878#0: *3 http wait request handler
2015/12/06 13:36:01 [debug] 19878#0: *3 malloc:<PHONE_NUMBER>DDDA50:3145728
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL_read: -1
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL_get_error: 2
2015/12/06 13:36:01 [debug] 19878#0: *3 free:<PHONE_NUMBER>DDDA50
2015/12/06 13:36:01 [debug] 19878#0: *3 http wait request handler
2015/12/06 13:36:01 [debug] 19878#0: *3 malloc:<PHONE_NUMBER>DDDA50:3145728
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL_read: 386
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL_read: -1
2015/12/06 13:36:01 [debug] 19878#0: *3 SSL_get_error: 2
2015/12/06 13:36:01 [debug] 19878#0: *3 reusable connection: 0
2015/12/06 13:36:01 [debug] 19878#0: *3 posix_memalign:<PHONE_NUMBER>D01F00:4096 @16
2015/12/06 13:36:01 [debug] 19878#0: *3 http process request line
2015/12/06 13:36:01 [debug] 19878#0: *3 http request line: "GET / HTTP/1.1"
2015/12/06 13:36:01 [debug] 19878#0: *3 http uri: "/"
2015/12/06 13:36:01 [debug] 19878#0: *3 http args: ""
2015/12/06 13:36:01 [debug] 19878#0: *3 http exten: ""
2015/12/06 13:36:01 [debug] 19878#0: *3 posix_memalign:<PHONE_NUMBER>DCF2D0:4096 @16
2015/12/06 13:36:01 [debug] 19878#0: *3 http process request header line
2015/12/06 13:36:01 [debug] 19878#0: *3 http header: "Host: apespace.org"
2015/12/06 13:36:01 [debug] 19878#0: 3 http header: "User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0 Iceweasel/38.4.0"
2015/12/06 13:36:01 [debug] 19878#0: 3 http header: "Accept: text/html,application/xhtml+xml,application/xml;q=0.9,/;q=0.8"
2015/12/06 13:36:01 [debug] 19878#0: *3 http header: "Accept-Language: en-US,en;q=0.5"
2015/12/06 13:36:01 [debug] 19878#0: *3 http header: "Accept-Encoding: gzip, deflate"
2015/12/06 13:36:01 [debug] 19878#0: *3 http header: "Cookie: PHPSESSID=qsuhbb0818ud3gbajf51r76oq6; jsAvailable=1"
2015/12/06 13:36:01 [debug] 19878#0: *3 http header: "Connection: keep-alive"
2015/12/06 13:36:01 [debug] 19878#0: *3 http header: "Cache-Control: max-age=0"
2015/12/06 13:36:01 [debug] 19878#0: *3 http header done
2015/12/06 13:36:01 [debug] 19878#0: *3 event timer del: 22:<PHONE_NUMBER>729
2015/12/06 13:36:01 [debug] 19878#0: *3 generic phase: 0
2015/12/06 13:36:01 [debug] 19878#0: *3 rewrite phase: 1
2015/12/06 13:36:01 [debug] 19878#0: *3 test location: "/"
2015/12/06 13:36:01 [debug] 19878#0: *3 test location: ~ ".(jpg|jpeg|gif|png|ico|css|js|htm|html|ttf|woff|svg)$"
2015/12/06 13:36:01 [debug] 19878#0: *3 test location: ~ ".(tpl|md|tgz|log|out)$"
2015/12/06 13:36:01 [debug] 19878#0: *3 test location: ~ ".php$"
2015/12/06 13:36:01 [debug] 19878#0: *3 test location: ~ "/."

Of course I no speak like you with no evidence and I will probe here . I make 3 HUB in one server every HUB I make in 3 different users apache2 for probe what resources they are broken and when . The 3 domains HUbs have users and working in the last test in 24 that I will post here is the resort. I not have crash because I have more ram and space of what Hubzilla can waste in 24 hours but any user here with a small VPS have many possibilities of have a  Installation Crash that is a fact not  imaginations or every one is round and you are right because you say. But I challenge you and any one here to tell me what is happens here:
HUB Number One Cache Directory  Process open in 24 hours and not close
/usr/bin/php include/onedirsync.php 857
All similar Lines using around 20% cpu each process number of open process around 75
Other Open Process
/usr/bin/php include/externals.php Open 3 times
/usr/bin/php include/poller.php   Open 1 time
Some process open few HOURS
HUB NUMBER 2 Normal Hubzilla Installation
Open Process 3 Times /usr/bin/php include/externals.php cpu use 17% to 7 % each process
HUB NUMBER 3 Normal Hubzilla Installation
Open Process 3 times /usr/bin/php include/externals.php seam similar use of CPU of Number 2
Other Open Process /usr/bin/php include/poller.php
Now I already see Tickets of similar problems process not close according to the Apache2 Setup you have in the INSTALLATION File . My MYsql not complaint I have in off memory Holding but Apache2 not look like at all Hubzilla Code . Any small VPS again sure will crash if start problems like that now there many tickets around all this and really I will like a explanation no hypothesis
I am really tired of hypothesis.

I believe you and many here try to blame always to servers memory ram etc of order and not accept there serious issues.

@royalterra just trying to help here that's all...
If you have issues please report them as new separate issues.
If you have fixes please make a pull request.
Please DO NOT spam other issues with your problems...

@git-marijus that is the correct reply you can't see all is relate to the crash of this man anyway all the Best possible is better start looking for alternatives .

you can't see all is relate

I can't at the moment... If you can you are invited to help out...

Is there a /var/log/php5-fpm.log or similar? Some nginx configurations throw errors into different files.
I have used nginx a lot, with great success, but never with PHP, so I'm a bit at sea here.
Somewhere there is a stacktrace that explains that 500 WSOD error. I'm confident that once it's located then it'll show clearly what the problem is.

There are some lines at the bottom of the supplied .htconfig.php file; which if uncommented will enable a PHP error log (extremely useful for finding the source of white screen failures). This isn't done by default due to potential issues with logfile ownership and write permissions and the fact that there is no logfile rotation by default.
@bashrc - are you using the api? Or do you get this error with any page access?

So I tried uncommenting those last lines, but the expected log was not produced. Whatever is happening here must be at a very fundamental level, such that it doesn't even get a chance to write a log. The good news is that it's completely reproducible.

Ok @bashrc have you try to make a git PULL back by example I got errors with last upgrade and I revert back to previous version.
Try to make a copy of all your files example with root user go to the root before yoru domain can be /var/www or what ever and execute:  cp -R myHUBdomain  securitycopy
Now we have a security copy of all files and folders in case anything crash more
Now I see you start this post 5 days ago . OK we will try to revert 6 days for be sure that make 8640 minutes. First stop apache or ngix what ever you use  go to the  folder of your installation
and execute us root: git reset --hard master@{"8640 minutes ago"}
now go back with
cd ..
execute: chown -R www-data:www-data yourfolderinstallation
I assume you are using user www-data if not use the ngix user you have
if you no have idea what user you have execute:
ls -l
and you will get something like
rwxr-xr-x  2 www-data www-data 4096 Dec  6 23:08 Backup
drwxr-xr-x  3 www-data www-data 4096 Dec  6 23:12 backup.com
that means the user is www-data and group www-data
Make sure you use the command chown with the correct user and group
Now START apache and try to see if your web is working
If is not working you can remove the folder myHUBdomain
and RENAME my securitycopy to the previous folder name you delete and use the command chown to assignate the correct user and group
And now you are again in the start point

Ensure that php.out can be created/written by the webserver uid (www-data in the case of apache; don't know what nginx uses). If it's not getting to PHP at all it's in the webserver, but that would mean that it should have no impact our code changes as PHP wouldn't even be running. As a real worst case debugging aid, change the php logging line to init_set('display_errors',1) and you should get the error onscreen.

It's not related to minutes and I know where the problem occurs. I went through commit by commit until it stopped working. ddce0412ac8fe675153182909d82955c79d1f660 is where it breaks, and it remains broken thereafter.

Ok @bashrc in that case in the seam happens to me before and at the end I finish making a new git pull with other folder name copy store in the new folder name and copy .htconfig.php and "chown" that folder
In apache I setup to look that folder and after all the git pull work. Today I have the seam problem with 2 git pull in 2 domains I can revert back but I can continue forward with pull . Really I am thinking to go back to the old method of rewrite files with ftp . Because now is 2 times in 2 months

In .htconfig.php I changed init_set('display_errors',1), but still nothing shown on screen when re-loading the site and no log produced. File ownership all looks ok, and if there was a problem with that I doubt I'd be able to revert to a working commit.

Ok here goes:
git checkout master
git pull
git branch test
git checkout test
git revert ddce0412ac8fe675153182909d82955c79d1f660
git commit -a

Reload the page, and it works.
This is definitely looking like a bug with that specific commit.

As pointed out, there's nothing special in that commit - it just moves an existing function to another file. However I've seen weird php errors with function/class name redefinition before that were very hard to get any debug output for. They tend to just core dump. I'm thinking it may be related to require_once('oauth.php')  - a file which exists (with different contents) at two places in the include path. My theory is that if the first include path is already loaded, it may cycle to the next include path - and get the wrong one and redefine the OAuth2 class. I've just added specific paths to this. Hopefully that makes a difference. As also noted, this isn't occurring with anybody else. I don't understand why it's specific to your site and wouldn't happen on other sites. I also don't know what's going on with royalterra's site since that site seems to throw fatal errors if you look at it funny.

If the latest git does not fix the issue, I'd be commenting out lines in api_auth.php until it comes back, or comment out everything and then bring back lines until it breaks.

Tested with the latest git and still doesn't work.

It looks like you're onto something. Making a new test branch from master, then commenting out:
require_once('include/oauth.php');

from include/api_auth.php, that works.  Remove the comment again and it fails. So that's the origin of the bug.

OK thanks - you may need to leave it commented until somebody figures out why. What's your php version? .

PHP 5.6.14-0+deb8u1 (cli) (built: Oct  4 2015 16:13:10)

Just a blind guess...
Instead of commenting out require_once('include/oauth.php'); in include/api_auth.php does commenting out require_once('include/api_auth.php'); in  include/auth.php make any difference?
The latter one is not actually needed i think and i am not sure why it was added in the affected commit anyway...

Yes that does also work: require_once('include/oauth.php') in api_auth.php and require_once('include/api_auth.php') commented out in auth,php

I mean only require_once('include/api_auth.php') in auth.php

We just need to make sure @kenrestivo can get his addons working  - otherwise this makes sense. We should only and specifically include the oauth and http login stuff when needed. We've got several different oauth implementations and even http-auth is duplicated between api and dav.
Ultimately we should also separate oauth from http-auth and standardise on a single oauth implementation and sort this mess out. For now let's just figure out what it takes to make it work again. I'd still like to know why it's only a problem for @bashrc - but since I can't duplicate it here, I can't trace it.

Can someone clarify what is the "it" is? i.e. what command or URL fails, and what is the failure?
If it's a break that can't be duplicated on anyone else's system, I don't see any reason to break working functionality (i.e. migration from RM to HZ), in order to chase something where no actual cause is yet found.
Agreed, however, that the whole API namespace and system needs to be sorted out. Unfortunately, I dumped 50 hours of time into writing the migratior, never got paid for it by the guy who wandered into the support channel and promised money, and don't have time to do much else on this, so it won't be me.

I've moved the oauth include inside the api_login() function and done a fair bit of work on api_login (and oauth) to undo "bit rot and rust", as these haven't really been touched in ages (4 years or so) and were a bit crufty. This may alleviate the problem completely or may reduce it to happening only on API login and in any event shouldn't affect the migrator at all. I'm going to tag it with 'retest please'.
It was never spelled out specifically but my feeling from earlier comments is that the entire app whitescreened - but from my reading of the code this would've only happened during login - and preventing login, as this is the only time we normally include auth.php. It would still be a nasty little bug. Anyway, I'm not planning anything else for the moment until we know how my latest work impacted the issue.

Pulling the latest on master, that works. Thanks.

I suspect a PHP module or PEAR module you've got on your system is having a namespace collision with.
One final test - Could you visit api/statuses/update and see if you get a login prompt or a white screen (?). That will tell me if the issue is with oauth1 or oauth2.

api/statuses/update shows a white screen

Thanks.

I think this issue is solved by now.


SPDX 3.0

Is anyone working on updating this to support SPDX 3.0?

I don't currently have the bandwidth for that, unfortunately.