text:
Allow Drawer width to be customized

Elevate the internal width parameter for the Drawer widget to the constructor, and default it to the previous constant value. More info on #99776 and #19724
Fixes #99776  and #19724
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

The way we typically resolve this kind of value is 1. individual instance, 2. DrawerThemeData, then 3. default.
Could you model this change after the other properties, e.g. backgroundColor?

Can do! Is it okay if I add the existing _kWidth default to DrawerThemeData?


text:
[image_picker] Fix path of returned File objects when picking videos

Description
Returning .description on a NSURL gave back an url that starts with file:///, when passing this url to the File object it constructed a path like /file:///foo/bar/baz.ext.
This fixes this logic to use .path instead, which returns the path to the file.
Here is some more background to how I found this: https://github.com/firebase/firebase-ios-sdk/pull/2458/files#r276273826
Related Issues
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Not sure how to add tests since the tests since to mock out the Objective-C part of the plugins, any help would be appreciated!

I tried manually testing this in the iOS simulator and unfortunately I'm getting "Error Loading Video"

I tried manually testing this in the iOS simulator and unfortunately I'm getting "Error Loading Video" (both before and after the change)

Just curious, how did you test camera with simulator?

I tried manually testing this in the iOS simulator and unfortunately I'm getting "Error Loading Video" (both before and after the change)

Just curious, how did you test camera with simulator?

I didn't, I just dragged and dropped a MOV file onto the simulator to get it into the videos library.

@LinusU Thank you for providing this. the .description returns the absolute path to the source. However .path seems to be more relevant to iOS file system and Apple seems suggesting to use .path.
Could you provide the use case that the absolute path failed? It seems that dart doesn't care which way the file path was expressed and works for both paths.
Also, it is probably a breaking change since the format of the returning value is changed, could you update the minor version? to 0.6.0?

Thank you for providing this. the .description returns the absolute path to the source. However .path seems to be more relevant to iOS file system and Apple seems suggesting to use .path.

Since this is an NSURL instance, it returns an absolute url, not path. So the problem is that it returned file:///foo/bar/baz, instead of /foo/bar/baz.
When creating a dart File with the string file:///foo/bar/baz, it will not create a file that points to /foo/bar/baz, but rather a file that points to /file:///foo/bar/baz, which won't work.

Could you provide the use case that the absolute path failed? It seems that dart doesn't care which way the file path was expressed and works for both paths.

I'm getting an error that Dart cannot read the file /file:///foo/bar/baz, I don't see how this can have ever worked, unless iOS 12.2 changed the behaviour of .description on NSURL.

Also, it is probably a breaking change since the format of the returning value is changed, could you update the minor version? to 0.6.0?

I don't think it's a breaking change since this works around an error. Even if description previously returned the file path without prepending file:/// in front of it, it shouldn't change now...
Hope this clears it up ☺
I'll rebase on master to fix the conflicts! 🚀

I'm getting an error that Dart cannot read the file /file:///foo/bar/baz, I don't see how this can have ever worked, unless iOS 12.2 changed the behaviour of .description on NSURL.

I just checked with the the current flutter, it is able to successfully read the file at file:///...., if you run the example app on image picker, does it pick up the file?

I can confirm this happens for us as well.  We have
File _stripUriScheme(File file) {
  if (file.path.startsWith("file:///")) {
    final filePath = file.path.substring(7);
    debugPrint('File URL pattern dropped... path is now: $filePath');
    return File(filePath);
  }
  // N.B. file://<some-host>/<some-path> is not handled
  // ...but I don't think that happens in practice
  return file;
}

as a workaround until this gets merged.
IIRC, it was as simple as running
File f = await ImagePicker.pickVideo(source: ImageSource.gallery);
file.lengthSync();

I can confirm this happens for us as well. We have
File _stripUriScheme(File file) {
  if (file.path.startsWith("file:///")) {
    final filePath = file.path.substring(7);
    debugPrint('File URL pattern dropped... path is now: $filePath');
    return File(filePath);
  }
  // N.B. file://<some-host>/<some-path> is not handled
  // ...but I don't think that happens in practice
  return file;
}

as a workaround until this gets merged.
IIRC, it was as simple as running
File f = await ImagePicker.pickVideo(source: ImageSource.gallery);
file.lengthSync();

@LinusU As commented here by @wreppun, people might be doing some surgery on the return value. So it is probably a breaking change for them. I still think we should bump the minor version and make it 0.6.0.

I agree, let's update the CHANGELOG with a description of new behavior and call it 0.6.0

Rebased and changed to breaking change 👌

source: gharchive/pull-request

authors: LinusU collinjackson cyanglaz wreppun

repo: flutter/plugins

license_type: permissive

license_source: github-api


text:
[image_picker] Add "DateTimeOriginal" to EXIF copied attributes.

Description
Adds DateTimeOriginal to EXIF copied image attributes. Since it corresponds to the time when the photo is taken, it is an important information.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[X] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[X] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[X] The analyzer (flutter analyze) does not report any problems on my PR.
[X] I read and followed the [Flutter Style Guide].
[X] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy].
[ ] I updated CHANGELOG.md to add a description of the change.
[X] I signed the [CLA].
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[X] No, this is not a breaking change.

(Hello, I am from China and my English is not good) In ios environment of flutter module, image_picker cannot be used. The first time you open it, there is nothing corresponding. The second time you directly report an error: Platform Exception (Multiple _ Request, Canceled by a Second Request, Null)

We apologize for the long delay in triaging this PR. We’re in the process of overhauling our PR triage system to respond much more quickly, as well as working through the backlog.
Per Flutter policy (and as indicated by the checklist) this will need tests before it can move forward.

Thank you for your contribution. I'm going to close this PR for now since there are outstanding comments, just to get this off our PR review queue. Please don't hesitate to submit a new PR if you have the time to address the review comments. Thanks!

authors: SuPeiSen aloisdeniel stuartmorgan


text:
[url_launcher] Add a new launchUrl to platform interface

This creates a new platform interface method for launching that closely
parallels the new public-facing API, so that implementations can switch
to implementing a more platform-neutral implementation. This will pave
the way for things like cleanly implementing
externalNonBrowserApplication support on non-iOS platforms.
To fully future-proof this method, it uses a wrapper parameter object
so that new types of parameters (i.e., things that aren't just new launch
modes or webview configuration) can be added to it without breaking
changes.
A follow-up will switch the app-facing package to call this new method
instead of the legacy method in the new APIs.
Implementation packages can adopt the new method as is useful for them;
eventually we can do a cleanup pass if we want to fully deprecate the
old method.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

I was wondering if the new API should be Uri-based, but it probably doesn't add much in the way of performance or security :/

It's non-obvious here because it doesn't include the second part (maybe I should have done it as a multi-package initial PR), but it can't be; launchUrlString needs to use this new method too, and we can't convert to Uri there because that API exists for cases where Uri can't handle a URL that the platform can.

Yes, yes, I remember the _string API for some of the funky non-standard URLs that some users needed!

authors: ditman stuartmorgan


text:
Error: No named parameter with the name 'animation'.             animation: animation,

flutter doctor -v
[✓] Flutter (Channel beta, v1.17.0, on Mac OS X 10.15.2 19C57, locale en-GB)
• Flutter version 1.17.0 at /Users/vishal.saini/Downloads/flutter
• Framework revision d3ed9ec945 (2 weeks ago), 2020-04-06 14:07:34 -0700
• Engine revision c9506cb8e9
• Dart version 2.8.0 (build 2.8.0-dev.18.0 eea9717938)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /Users/vishal.saini/Library/Android/sdk
• Platform android-29, build-tools 28.0.3
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.2.1, Build version 11B500
• CocoaPods version 1.8.4
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 3.4)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 39.0.1
• Dart plugin version 183.6270
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
[✓] IntelliJ IDEA Ultimate Edition (version 2018.3.4)
• IntelliJ at /Applications/IntelliJ IDEA.app
• Flutter plugin version 39.0.1
• Dart plugin version 183.6270
[✓] Connected device (3 available)
• iPhone 6   • c0445b8f5e0548712bc1c733c6826fb8f39ac13f • ios            • iOS 12.4.4
• Chrome     • chrome                                   • web-javascript • Google Chrome 81.0.4044.113
• Web Server • web-server                               • web-javascript • Flutter Tools
• No issues found!

https://github.com/fluttercandies/extended_image/issues/167

source: gharchive/issue

license: MIT


text:
你们dio库是怎么封装的？

dio网络库。

@cikichen https://github.com/flutterchina/dio
:octocat: From gitme Android

@wendux 可能表达不准确，我的意思是dio的工具类封装

创建一个全局dio单例，所有请求都通过这个单例来发送
:octocat: From gitme Android

authors: cikichen wendux

repo: flutterchina/gitme


text:
Added onExpand onCollapse callbacks

I find it very useful to know when the front layer expands or collapses. It might come in handy if you want to change or do smt reactively. In my case, I needed to change toolbar title when state of frontLayer changes, so I added this feature.

@WieFel Please review this? I think this feature is useful.

I doubt if onFrontLayerVisible is an appropriate variable name. onExpanded in my opinion is way more familiar and intuitive, also it's unclear what onFrontLayerVisible means as front layer is always visible at least partly. I'll rename variables, but write if u changed ur mind.

@vlytvyne We already expose showFrontLayer and showBackLayer helper methods with Backdrop.of(context), then nomenclature is to keep consistency with that.
Yes it is not the most appropriate. But if we are planning to change the names then it should be everywhere - keeping backward compatibility in mind
I would say following names would be most appropriate.

showFrontLayer -> backdropClose
showBackLayer -> backdropOpen
onFrontLayerVisible -> onBackdropClose
onBackLayerVisible -> onBackdropOpen

Please note that "backdrop" it's a composition of both front and back layers so backdropOpen and similar variants are puzzling.
From  :
"A backdrop is composed of two surfaces: a back layer and a front layer. The back layer displays actions and context, and these control and inform the front layer's content."

@vlytvyne
MDG uses following terms for this - "Revealing the back layer" and "Concealing the back layer".
Keeping that in mind, the name should be as following:

showFrontLayer -> concealBackLayer
showBackLayer -> revealBackLayer
onFrontLayerVisible -> onBackLayerConcealed
onBackLayerVisible -> onBackLayerRevealed

I definitely like it better this way.

@WieFel Let us know your views.

@vlytvyne @WieFel I have done the changes. Please check.

@WieFel Yeah, you are right.

@WieFel Done requested changes.

This feature have landed in pub.dev with v0.3.3.
@vlytvyne Check yourself out in Contributors Section in README.md!!

authors: daadu vlytvyne

repo: fluttercommunity/backdrop


text:
connectivity_plus always showing ConnectivityResult.wifi on web, even when WiFI is switched off

System info
Issue occurs on: Web
Plugin name: connectivity_plus
Plugin version: 1.0.6
Steps to Reproduce
Running on Chrome browser, when connectivity state changes (I toggle Wifi), onConnectivityChanged() get called every time, but the enum value of ConnectivityResult always returns ConnectivityResult.wifi.
I tested the same in android, it works fine (shows ConnectivityResult.none while WiFI is turned off)

I agree with you. It doesn't matter if you are connected to wifi or not, it always returns ConnectivityResult.wifi

Same issue here.

What does the live output print?
https://googlechrome.github.io/samples/network-information/

I'm not sure why NetworkInformation.effectiveType is preferred over NetworkInformation.type, and why 4g is considered as... WiFi :eyes:
https://github.com/fluttercommunity/plus_plugins/blob/main/packages/connectivity_plus/connectivity_plus_web/lib/src/utils/connectivity_result.dart

authors: danieldoddjr daniellampl jpnurmi rahatbs23

license_source: bigquery


text:
Image update automation appending instead of replacing

Hello, I'm experiencing issues with image update automation where the latest tag is being appended to the image tag instead of replacing it.
version: ghcr.io/fluxcd/image-automation-controller:v0.22.1
The tag in the original manifest is gcr.io/xyz/crasample:develop-deadbeef-0, the latest tag is gcr.io/xyz/crasample:develop-deadbeef-3 and the automation result is gcr.io/xyz/crasample:develop-deadbeef-0:develop-deadbeef-3
When the manifest was gcr.io/xyz/crasample:5.0.0 the result was gcr.io/xyz/crasample:5.0.0:develop-deadbeef-3
apiVersion: image.toolkit.fluxcd.io/v1beta1
kind: ImagePolicy
metadata:
  name: crasample-crasample
  namespace: some-namespace
spec:
  filterTags:
    extract: $ts
    pattern: ^develop-[a-fA-F0-9]+-(?P<ts>.*)
  imageRepositoryRef:
    name: crasample-crasample
    namespace: convai
  policy:
    numerical:
      order: asc

apiVersion: apps/v1
kind: Deployment
spec:
  ...
      containers:
      - image: gcr.io/nosidelines/crasample:develop-deadbeef-0 # {"$imagepolicy": "some-namespace:crasample-crasample"}

~ flux get all -n some-namespace
NAME                               	LAST SCAN                	SUSPENDED	READY	MESSAGE
imagerepository/crasample-crasample	2022-06-07T13:57:16-10:00	False    	True 	successful scan, found 61 tags

NAME                           	LATEST IMAGE                                                      	READY	MESSAGE
imagepolicy/crasample-crasample	gcr.io/xyz/crasample:develop-deadbeef-0:develop-deadbeef-3	True 	Latest image tag for 'gcr.io/xyz/crasample:develop-deadbeef-0' resolved to: develop-deadbeef-3

NAME                                	LAST RUN                 	SUSPENDED	READY	MESSAGE
imageupdateautomation/team-manifests	2022-05-19T13:41:00-10:00	False    	False	encountered invalid image ref "gcr.io/xyz/crasample:develop-deadbeef-0:develop-deadbeef-3": could not parse reference: gcr.io/xyz/crasample:develop-deadbeef-0:develop-deadbeef-3

I've managed to reproduce this, the issue comes from setting a tag in the ImageRepository, starting with next version we'll error out when tags are used in .spec.image.

thank you. you were right about the tag in the ImageRepository

repo: fluxcd/image-reflector-controller


text:
Allow wildcard value for namespaces in Alert eventSources

Allow wildcard value for namespaces in Alert eventSources.
Fix: https://github.com/fluxcd/notification-controller/issues/71
At the moment we have to create a list of all HelmRelease objects, for example, in the eventSources section to be able to get notifications about them. We also need to update the list whenever a new object is added. That requires lots of effort. It seems that the best way is to allow wildcards for namespaces.
How the Alert eventSources definition looks now:
  eventSources:
    - kind: HelmRelease
      name: '*'
      namespace: 'service1'
    - kind: HelmRelease
      name: '*'
      namespace: 'service2'
...
    - kind: HelmRelease
      name: '*'
      namespace: 'service99'

How it would be after the change:
  eventSources:
    - kind: HelmRelease
      name: '*'
      namespace: '*'

There are security concerns related to this feature that were articulated in #71 (e.g. https://github.com/fluxcd/notification-controller/issues/71#issuecomment-991049078). Given the mentioned RFC has never been merged, we need to think this over.
In any case this will need to be held until notification-controller 1.0.0 has been released. Let's please take the time to completely discuss the threat model and potential mitigations here.

I thought that disabling cross-namespace references should solve the security concern as was said there:
https://github.com/fluxcd/notification-controller/issues/71#issuecomment-1374690404. I agree that using ReferenceGrant should improve the security boundaries. Still, it seems to be orthogonal to the Alerts setup. notification-controller can listen to events in any namespace anyway if the cross-namespace references flag is enabled. If this is inappropriate cluster admin can disable cross-namespace references.
Are there any ongoing discussions about the threat model? I see that https://github.com/fluxcd/flux2/pull/2092 and https://github.com/fluxcd/flux2/pull/2093 were closed both to wait for https://github.com/kubernetes/enhancements/blob/master/keps/sig-auth/3766-referencegrant/README.md

any update on this?

Hi! Is it correct to say that this PR is still on hold and we are waiting?

Hi! Is it correct to say that this PR is still on hold and we are waiting?

Yes, pretty much. Current consensus is that we want to wait for the Kubernetes ReferenceGrant API to become GA which will definitely take a couple of Kubernetes releases to come.

This feature request seems irrelevant to ReferenceGrant API; what ReferenceGrant will affect is all of the cross-namespace references filtering - but that'd have to work at a different level. I see no point in holding on with this PR.
The current model is pretty secure - this PR does not make it weaker.

Hi, any update on this PR?

authors: MOZGIII alekspog gecube makkes rinzler-17 xTrekStorex

repo: fluxcd/notification-controller


text:
Instructions to install on server and webp

Hi,
I'm looking for simple instructions to install flyimg on my server and wonder if webp is supported.
Regards & Thanks,
David

Hi @Allmedialab ,
Thank you for opening the issue,
we provide simple documentation on how you can run Flyimg using docker on any host.
If you want to have a self-installed version, you can check the Dockerfile in the main folder but be sure you have all dependencies installed in your server (that's why we provide a ready docker image).
And yes Webp is supported by default
Thank you again!
Best,
Sadok

authors: Allmedialab sadok-f

repo: flyimg/flyimg

license: mit


text:
demo installation fails at make init

Following the instructions here: https://flynn.io/docs/installation/vagrant, I am getting this output after typing make init:
CLUSTER_DOMAIN="demo.localflynn.com" SSH_AUTH_SOCK="" FLAG_BOOTSTRAP=false vagrant box update
==> default: Box 'flynn-base' not installed, can't check for updates.
CLUSTER_DOMAIN="demo.localflynn.com" SSH_AUTH_SOCK="" FLAG_BOOTSTRAP=false vagrant up --provider virtualbox
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box 'flynn-base' could not be found. Attempting to find and install...
    default: Box Provider: virtualbox
    default: Box Version: > 0
==> default: Box file was not detected as metadata. Adding it directly...
You specified a box version constraint with a direct box file
path. Box version constraints only work with boxes from Vagrant
Cloud or a custom box host. Please remove the version constraint
and try again.
make[1]: *** [up] Error 1
make: *** [init] Error 2

If I comment out the line config.vm.box_version = "> 0" from the Vagrantfile, then I get a different error message:
CLUSTER_DOMAIN="demo.localflynn.com" SSH_AUTH_SOCK="" FLAG_BOOTSTRAP=false vagrant box update
==> default: Box 'flynn-base' not installed, can't check for updates.
CLUSTER_DOMAIN="demo.localflynn.com" SSH_AUTH_SOCK="" FLAG_BOOTSTRAP=false vagrant up --provider virtualbox
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box 'flynn-base' could not be found. Attempting to find and install...
    default: Box Provider: virtualbox
    default: Box Version: >= 0
==> default: Box file was not detected as metadata. Adding it directly...
==> default: Adding box 'flynn-base' (v0) for provider: virtualbox
    default: Downloading: https://dl.flynn.io/vagrant/flynn-base.json
An error occurred while downloading the remote file. The error
message, if any, is reproduced below. Please fix this error and try
again.

make[1]: *** [up] Error 1
make: *** [init] Error 2

I installed the latest Virtualbox (5.1.8) and Vagrant (1.8.7) on Mac OS X 10.11.4.
I am pretty new on this, so there might be something I'm doing wrong, but I believed I did everything pretty much as explained.

@jperals the output is indicating that Vagrant could not download the manifest file at the following URL:
https://dl.flynn.io/vagrant/flynn-base.json
That link works for me, do you get valid JSON when you go to that URL?

Looks like the latest version of Vagrant is broken with our Vagrantfile. I'm investigating.

@lmars Yes, I also get good looking JSON when I visit that URL with my browser (and on the same network as when using the failing scripts).

This is an upstream issue: mitchellh/vagrant#7970
Looks like this workaround works:
sudo rm /opt/vagrant/embedded/bin/curl

@titanous Right! In my case I applied the symbolic link workaround:
sudo ln -s /usr/bin/curl /opt/vagrant/embedded/bin/curl
and it worked as well.
Thanks!

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.

authors: jperals lmars titanous

repo: flynn/flynn


text:
migrate-domain timed out

Hello,
A new flynn tester here.
I managed to install flynn on three hosts in our test vmware infrastructure.
I wanted to change the dns to something else, here is the command I ran:

flynn cluster migrate-domain paas_test.my_company.org

Migrate cluster domain from "6lmy.flynnhub.com" to "paas_test.my_company.org"? (yes/no): yes
Migrating cluster domain (this can take up to 2m0s)...
timed out waiting for job events: map[scheduler:map[up:3]]
timed out waiting for job events: map[scheduler:map[up:3]]
timed out waiting for domain migration to complete
Before running this command, we have registered paas_test.my_company.org in our internal DNS (pointing to the three host IP addresses).
Could you guide us on this? As I'm new, what's the procedure to give you the right logs?
PS: If it succeed, will it remove the 6lmy.flynnhub.com entry in your DNS?
Tks!

Please run sudo flynn-host collect-debug-info on the server.

sorry, I'm not able to submit any logs. the command always freeze at:
INFO[12-19|11:12:46] getting job logs
even if I wait for hours.
The list command shows well my three hosts.
The fix command blocks as well. I think I have to reinstall everything.

You could also manually collect the logs from /var/log/flynn and send a tarball to logs@flynn.io

Closing due to lack of information.


text:
Missing PGPORT env var

After adding a postgres resource our app was expecting PGPORT to be an available environment port. This however wasn't the case. Can you add PGPORT to the exported environment variables?

I'm not aware of any standard software that requires PGPORT when it is the default of 5432, like it is on Flynn. Can you just set it for your app?

It's likely a very rarely used env var, but flynn does provide all the env vars necessary to reconstruct the DATABASE_URL except the port. Is it reasonable to always have apps hard code the postgres port?
I'm inclined to suggest to not have apps hard code the postgres port since all the other env vars necessary to construct a new DATABASE_URL are being exported except PGPORT. This is coming from more of an app developer point of view though.
It looks like Heroku doesn't export anything but DATABASE_URL (I could be wrong), so an app doesn't have the option. Flynn gives you the option to use DATABASE_URL or to rely on standard postgres env vars.

I'm not saying to hardcode it, just default to 5432 (the PostgreSQL port registered with IANA) if PGPORT isn't set. If you don't specify a port, that's what libpq (the official client) and most other clients I've seen use.

Yeah good point. If not specified then assume the port is 5432. I hadn't realized that this port was a full-on standard.

authors: dschnare titanous


text:
deployer: Support waiting for service discovery events

Closes #849.

@titanous comments addressed

LGTM pending minor comment.

authors: lmars titanous


text:
Add Jaehyun's link

안녕하세요~ 😀

추가해주셔서 감사합니다! ㅎㅎ 반갑습니다

authors: flynnpark jaehyun-study

repo: flynnpark/nomadcoders-github-list


text:
keystonev3 allow to pass user_domain differing auth domain

e.g. my user lives in the default domain, but I can still scope my token to my second 'custom' domain :)

@Carthaca,
You're right, the user domain is needed.
And along #7 to handle more cases!
Thanks

authors: Carthaca gildub

repo: flystack/misty


text:
#3458 Include default MinIO credentials

#3458 Include default MinIO credentials in sandbox console readout
TL;DR
Include default MinIO credentials in sandbox console readout
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
When following the steps to setup Flyte on a demo sandbox, the console reports that 📂 The Minio API is hosted on localhost:30002. Use http://localhost:30080/minio/login for Minio console. However, in order to view the obect store, you need to use the default credentials for MinIO, which are:
username: minio
password: miniostorage
Tracking Issue
https://github.com/flyteorg/flyte/issues/3458

+1 to this PR...I had to decode the secrets to get these credentials. Maybe put these in Constants in the same file than hard coding in the message?

repo: flyteorg/flytectl


text:
SQL SERVER run cursor script stop and no exception throw out

What version of Flyway are you using?
4.0.3
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
Maven plugin
What database are you using (type & version)?
SQL SERVER 2014
What operating system are you using?
OSX 10.11.6
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
Execute Script
EXEC dbo.sp_executesql @statement = N'DECLARE @pkName VARCHAR(100)
    DECLARE @tableName VARCHAR(100)
    DECLARE @uqName VARCHAR(100)
    DECLARE @fkName VARCHAR(100)
    DECLARE @colName VARCHAR(100)
    DECLARE @schemaName VARCHAR(100)
    DECLARE @PK_CURSOR as CURSOR
    DECLARE @FK_CURSOR as CURSOR
    DECLARE @SQL VARCHAR(MAX)
    DECLARE @newPKName VARCHAR(100)

    SET @PK_CURSOR = CURSOR FOR
    SELECT tableName=tt.name,pk.name AS ''pkName''
    FROM sys.key_constraints pk
    INNER JOIN sys.tables tt ON tt.object_id = pk.parent_object_id AND tt.type=''U''
    WHERE pk.type=''PK'' --AND CHARINDEX(''_'',RIGHT(pk.name,17))=1
    ORDER BY tableName

    OPEN @PK_CURSOR;
    FETCH NEXT FROM @PK_CURSOR INTO  @tableName,@pkName

    WHILE @@FETCH_STATUS = 0
    BEGIN
      SET @SQL = ''sp_RENAME '''''' + @pkName + '''''' , ''''PK_'' + @tableName + ''''''''
      --PRINT (@SQL)
      EXEC (@SQL)
      FETCH NEXT FROM @PK_CURSOR INTO  @tableName,@pkName
    END

    CLOSE @PK_CURSOR
    DEALLOCATE @PK_CURSOR

    SET @FK_CURSOR = CURSOR FOR
    SELECT tableName=tt.name,pk.name AS ''uqName''
    FROM sys.key_constraints pk
    INNER JOIN sys.tables tt ON tt.object_id = pk.parent_object_id AND tt.type=''U''
    WHERE pk.type=''UQ''
    ORDER BY tableName

    OPEN @FK_CURSOR;
    FETCH NEXT FROM @FK_CURSOR INTO @tableName,@uqName

    WHILE @@FETCH_STATUS = 0
    BEGIN

      SET @SQL = ''ALTER TABLE '' + @tableName + '' DROP CONSTRAINT '' + @uqName
      --PRINT (@SQL)
      EXEC (@SQL)
     FETCH NEXT FROM @FK_CURSOR INTO @tableName,@uqName
    END

    CLOSE @FK_CURSOR
    DEALLOCATE @FK_CURSOR

    SET @FK_CURSOR = CURSOR FOR
    SELECT QUOTENAME(SCHEMA_NAME(o.[schema_id])) AS schemaName
      ,QUOTENAME(o.name)  AS tableName
      ,(i.name) AS indexName
      FROM sys.indexes AS i
      INNER JOIN sys.tables AS o
      ON i.[object_id] = o.[object_id]
    WHERE i.is_primary_key = 0
    AND i.index_id <> 0
    AND o.is_ms_shipped = 0;

    OPEN @FK_CURSOR;
    FETCH NEXT FROM @FK_CURSOR INTO @schemaName, @tableName,@colName

    WHILE @@FETCH_STATUS = 0
    BEGIN

      SET @SQL = ''DROP INDEX '' + @schemaName + ''.'' + @tableName + ''.'' + @colName
      --PRINT (@SQL)
      EXEC (@SQL)
     FETCH NEXT FROM @FK_CURSOR INTO @schemaName, @tableName,@colName
    END

    CLOSE @FK_CURSOR
    DEALLOCATE @FK_CURSOR

    SET @FK_CURSOR = CURSOR FOR
    SELECT
       f.name AS fkName,
       OBJECT_NAME(f.parent_object_id) TableName,
       COL_NAME(fc.parent_object_id,fc.parent_column_id) ColName
    FROM
       sys.foreign_keys AS f
    INNER JOIN
       sys.foreign_key_columns AS fc
        ON f.OBJECT_ID = fc.constraint_object_id
    INNER JOIN
       sys.tables t
        ON t.OBJECT_ID = fc.referenced_object_id
    --WHERE LEN(REPLACE(f.name,''FK_'',''''))=25

    OPEN @FK_CURSOR;
    FETCH NEXT FROM @FK_CURSOR INTO @fkName, @tableName,@colName

    WHILE @@FETCH_STATUS = 0
    BEGIN
    SET @SQL = ''IF EXISTS (SELECT * FROM sys.foreign_keys WHERE name = '''''' + @fkName + '''''' AND parent_object_id = object_id('''''' + @tableName + '''''')) '' +
           ''ALTER TABLE '' + @tableName + '' DROP CONSTRAINT '' + @fkName + '' END GO''

      SET @SQL = ''ALTER TABLE '' + @tableName + '' DROP CONSTRAINT '' + @fkName
      --PRINT (@SQL)
      EXEC (@SQL)
     FETCH NEXT FROM @FK_CURSOR INTO @fkName, @tableName,@colName;
    END

    CLOSE @FK_CURSOR
    DEALLOCATE @FK_CURSOR

    SET @PK_CURSOR = CURSOR FOR
    SELECT tableName=tt.name,dc.name AS dcName
    FROM sys.default_constraints dc
    INNER JOIN sys.tables tt ON tt.object_id = dc.parent_object_id AND tt.type=''U''
    WHERE dc.type=''D'' --AND CHARINDEX(''__'',RIGHT(dc.name,10))=1
    ORDER BY tableName

    OPEN @PK_CURSOR;
    FETCH NEXT FROM @PK_CURSOR INTO  @tableName,@pkName

    WHILE @@FETCH_STATUS = 0
    BEGIN
      SET @SQL = ''ALTER TABLE '' + @tableName + '' DROP CONSTRAINT '' + @pkName
      --PRINT (@SQL)
      EXEC (@SQL)
      FETCH NEXT FROM @PK_CURSOR INTO  @tableName,@pkName
    END

    CLOSE @PK_CURSOR
    DEALLOCATE @PK_CURSOR'

GO

What did you expect to see?
EXECUTE SUCCESS!
What did you see instead?
NO EXCEPTION

Please explain clearly what isn't working properly and reduce your script to the absolute smallest part that demonstrates the issue.

Closing due to lack of response from original reporter.

authors: axelfontaine mwylaoma

repo: flyway/flyway


text:
Flyway-command 9.21.1 has invalid tar in maven

Which version and edition of Flyway are you using?
Flyway-command 9.21.1
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Commandline via npx
Which database are you using? (Type & version)
postgres
Which operating system are you using?
linux x64 (ubuntu)
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
npx flyway -c db/config.js migrate
What did you expect to see?
What did you see instead?
$ npx flyway -c db/config.js migrate
Loading config from:  /home/project/config/local.env
DOWNLOADING https://repo1.maven.org/maven2/org/flywaydb/flyway-commandline/maven-metadata.xml
DOWNLOADING https://repo1.maven.org/maven2/org/flywaydb/flyway-commandline/9.21.1/flyway-commandline-9.21.1-linux-x64.tar.gz
/home/project/node_modules/node-flywaydb/bin/flyway.js:76
throw new Error(err);
^
Error: Error: aborted
at /home/project/node_modules/node-flywaydb/bin/flyway.js:76:19
at processTicksAndRejections (node:internal/process/task_queues:96:5)
$ tar -zxf /home/project/node_modules/node-flywaydb/jlib/flyway-commandline-9.21.1-linux-x64.tar.gz
gzip: stdin: unexpected end of file
tar: Unexpected EOF in archive
tar: Unexpected EOF in archive
tar: Error is not recoverable: exiting now

Actually - the file seems ok. Perhaps the process tries to untar before it's finished downloading. If i download and place under jlib it works


text:
Add classpath scanning support for GraalVM native images

Images compiled ahead of time by GraalVM's native-image tool cannot be opened as jar files. Instead, GraalVM introduces special file system implementation that allows accessing resources built in the image at compile time. Use this file system implementation to scan for migrations on classpath when Flyway compiled in as part of native image build.
See #2927

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


text:
Combine REST with Socket protocol in microservice architecture

hi, firstly, i must say this is super cool project, and i very love it. But i have some confuse, if you have time, please explain clearly for me.
i want to build a microservice, Client will call to one main service by HTTP, and then main service call to another service by message (websocket). But how i can skip the HTTP request and wait the message go to all other services, get the response and send back to to client :(
i want to use websocket to communication between internal microserivce because it's faster than HTTP protocol
many thank guys

@cjus do you have any idea about it? :D many thank

@tybq93 since this isn't but rather a question about architecture I invite you to join our slack channel. While you're considering that I can say that HTTP and messages (socket) can be combined at will. You can have an HTTP endpoint which in turns sends another service a message via hydra.sendMessage.
Hydra doesn't offer a messaging synchronization feature to keep track of messages sent and acknowledged. Rather it simply offers a message transmission and routing. To use socket only messaging you can create hydra message listeners in your services:
hydra.on('message', (message) => {
  // handle message / dispatch message
  this.myMessageHandlerAndDispacher(message);
});

When a message is received and you'd like to reply to it you can use hydra.sendMessageReply()
Also, you mentioned that you'd like to use WebSockets. Is that via HydraRouter or some other means?

yeah, i mean websocket is via HydraRouter.
for example:
step 1: client ----call http api to get list post---> API Service
step 2: API Service ----sendMessage get list post to PostService ----> PostService
step 3: PostService  --- sendMessageReply back to APIService -----> API Service
step 4: API Service ---send result back----> Client
but the problem here is we can not delay the step 1 request and wait util we have the response with result back on step 3 and then send back to user in step 4, so i think this is impossible, right?
I already sent you email to join slack channel, i'm very happy to discuss more about your team. :+1:

Correct that doesn't sound possible. It's an issue of whether you close the HTTP connection in step one or leave it open pending a response.  The way we do this is that we create websocket connections to HydraRouter (from mobile devices) and then our microservices able to route message responses back out through hydraRouter to specific devices.  This isn't fully documented and I need to do a write-up for anyone else wanting to do the same.
I sent you an invite to our Slack channel. See you there.

repo: flywheelsports/hydra


text:
Equalizer?

Does anyone know how to work with EQ?

Equalizer control is not currently implemented in this library – but we could add it.
However, this is rather complicated since it's 3 parameters (freq, q, gain) that can be controlled independently. In the end it's just numeric values so no big deal.
I'd just implement this if it's really necessary, though.

You can actually send raw messages to the mixer:
const ui = new SoundcraftUi(ip);
// connect and wait …

ui.conn.sendMessage(message);

The EQ messages are structured like this:
SETD^i.5.eq.b1.freq^0.330531810826151
SETD^i.5.eq.b1.q^0.510485569690792
SETD^i.4.eq.b1.gain^0.5165745856353591

i.5: Input 5
b1: EQ Point 1
value: between 0 and 1. This range has to be transformed into the actual fader/frequency range!

repo: fmalcher/soundcraft-ui


text:
undefined method `initialized? for backend:chain

@Sija @fnando @DefV
Rails 4.2.1, use Asset Pipeline, I did the list of things

add gem "i18n-js", ">= 3.0.0.rc11" to gemfile and bundle install
add i18n,  i18n/translations to application.js
and in the document, it tells me that the default setting is working. but it do not

chuck@chuck-MacBook-Pro:~/seaify/weishop(add_i18n_js|✚7…) % rake i18n:js:export
rake aborted!
NoMethodError: undefined method `initialized?' for #<I18n::Backend::Chain:0x007fc76789bb70>
/Users/chuck/.rvm/gems/ruby-2.1.5/gems/i18n-js-3.0.0.rc11/lib/i18n/js.rb:155:in `block in translations'
/Users/chuck/.rvm/gems/ruby-2.1.5/gems/i18n-js-3.0.0.rc11/lib/i18n/js.rb:154:in `instance_eval'
/Users/chuck/.rvm/gems/ruby-2.1.5/gems/i18n-js-3.0.0.rc11/lib/i18n/js.rb:154:in `translations'
/Users/chuck/.rvm/gems/ruby-2.1.5/gems/i18n-js-3.0.0.rc11/lib/i18n/js.rb:88:in `translation_segments'
/Users/chuck/.rvm/gems/ruby-2.1.5/gems/i18n-js-3.0.0.rc11/lib/i18n/js.rb:34:in `export'
/Users/chuck/.rvm/gems/ruby-2.1.5/gems/i18n-js-3.0.0.rc11/lib/tasks/export.rake:5:in `block (3 levels) in <top (required)>'
/Users/chuck/.rvm/gems/ruby-2.1.5/bin/ruby_executable_hooks:15:in `eval'
/Users/chuck/.rvm/gems/ruby-2.1.5/bin/ruby_executable_hooks:15:in `<main>'
Tasks: TOP => i18n:js:export
(See full trace by running task with --trace)

Could you tell me how to solve this? or write a wiki for this situation.
According to https://github.com/fnando/i18n-js/issues/59,  it seems that i should add a initializer file to set the default backend?

What i18n backend you are using?

@PikachuEXE
I use https://github.com/spree-contrib/spree_i18n this gem, not the rails-i18n.
I use i18n as keyword to search all the files, doesn't see where already set the backend for using.
So i guess the spree_i18n should defined a backend and use it.

I think this gem only support simple backend at the moment.
Try setting I18n.backend yourself to see what happen.

@PikachuEXE
I checkout the spree_i18n source code, and find this
I18n.backend = I18n::Backend::Chain.new(I18n::Backend::I18nDataBackend.new, I18n.backend)

so simplely set
 I18n.backend = I18n::Backend::Simple.new

won't work
you mean that the i18n-js currently can't support other backend?
Or how can i make the spree_i18n backend to support it?
what kind of things should be done?
Is there some method should be filled in other backend?

won't work <= Do you have the same error message again?
I want to know if the part I18n::Backend::Chain in for #<I18n::Backend::Chain:0x007fc76789bb70>
has changed.

@PikachuEXE
After set the backend as I18n::Backend::Simple.new, the error disappear, and i can use the i18n in coffeescript.
But in this project, i think it's not a solution in my project to simply replace the backend, i should fix the spree_i18n backend issue. It might bring some strange errors.

I don't I think I have time to handle non Simple backend as I18n.backend in 3.0.
And non Simple backend might work differently than Simple backend, which is the reason of assuming Simple backend in this gem.

@PikachuEXE
Ok. Got it. Understand. I will try to fix it my own, let it both works.
Thanks so much for your kindly reply!!!

Great :)
Feel free to chat at https://gitter.im/fnando/i18n-js

@PikachuEXE
can't i18n-js support backend:chain? I can put the I18n::Backend::Simple.new into to the chain. it can iterate the backend, and finally use the I18n::Backend::Simple.new.

The real problem is not trying to call init on simple backend, but instead unable to predict that how other backends in the chain causes the gem to have strange behaviour (i.e. bug).
I admit there should be a proper error for non-simple backend. I will do it before 3.0

hey @seaify ! are your changes still working well?? (I18n.backend = I18n::Backend::Simple.new
) i have the same problem with spree_i18n. Please let me know if you encountered another solution. thnks

Should this issue be closed? seems to me that it is still unresolved.

authors: PikachuEXE cjcruz maca seaify


text:
Algo Counting Sort Java

Here is the description of the algo
https://www.geeksforgeeks.org/counting-sort/

I'll take care of this!

@shanky199912 : Thanks for sharing

repo: fnplus/interview-techdev-guide


text:
🛑 Contaboo is down

In d6ba7f6, Contaboo (https://contabo.com) was down:

HTTP code: 403
Response time: 438 ms

Resolved: Contaboo is back up in c8f6973 after 3 hours, 43 minutes.

repo: fnzv/status


text:
🛑 Contaboo is down

In 4750fce, Contaboo (https://contabo.com) was down:

HTTP code: 403
Response time: 524 ms

Resolved: Contaboo is back up in 94089e3 after 49 minutes.


text:
🛑 Contaboo is down

In 038ff2a, Contaboo (https://contabo.com) was down:

HTTP code: 403
Response time: 530 ms

Resolved: Contaboo is back up in dcd1064 after 55 minutes.


text:
UIINREACH - 191 - components are incorrectly imported directly from stripes-* packages

Purpose
Issue : components are incorrectly imported directly from stripes-* packages
https://issues.folio.org/browse/UIINREACH-191

Please remove code smells.

@Dmytro-Melnyshyn
I have removed the code smells.
could you please review now?

authors: Dmytro-Melnyshyn Terala-Priyanka

repo: folio-org/ui-inn-reach


text:
bug: checker run on every startup ignoring checker frequency

Did you check docs and existing issues?

[X] I have read all the lazy docs
[X] I have searched the existing issues of lazy
[X] I have searched the exsiting issues of the plugin I have a problem with

Neovim version (nvim -v)
NVIM v0.9.0-dev-cb9d68fe
Operating system/version
MacOS 10.15.7
Describe the bug
Lazy check for plugins update on every neovim startup disregarding check.frequency value.
Steps To Reproduce

Start nvim and quickly open Lazy window
Easier to see if you have large amount of plugins and set checker.concurency to low number
You'll see Lazy checking every plugins fetch shown in Lazy window

Expected Behavior
Lazy will only check for update every 2 hours
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "--single-branch",
    "https://github.com/folke/lazy.nvim.git",
    lazypath,
  })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
  checker = {
  -- automatically check for plugin updates
  enabled = true,
  concurrency = 1, ---@type number? set to 1 to check for updates very slowly
  notify = false, -- get a notification when new updates are found
  frequency = 3600*2, -- check for updates every hour
  },
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

the frequency is a value used while neovim is running, not between all checks

fixed by https://github.com/folke/lazy.nvim/commit/813fc944d797fe1b43abe12866a9ef7af403c35c

authors: folke kuntau


text:
bug: helix generates invalid toml files

Did you check docs and existing issues?

[X] I have read all the tokyonight.nvim docs
[X] I have searched the existing issues of tokyonight.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
helix 24.3 (2cadec0b)
Operating system/version
macOS 14.5
Describe the bug
The extras/helix have invalid toml files with no values:
"markup.link" =
"markup.link.label" =
"markup.link.text" =
"markup.list" =
"markup.raw" =
"markup.raw.block" =

Steps To Reproduce

Copy the file to .config/helix/themes/tokyonight_storm.toml
Set :theme tokyonight_storm

Expected Behavior
Those lines should include a value or be removed if null
Repro
No response

I don't use helix and the code that the original OP wrote for it is pretty messy. Feel free to provide a PR

authors: AbeEstrada folke


text:
Update data/ForC_measurements.csv

Finished updating Chave_2008 data

I'm not sure why it can't be merged?

It looks like @ValentineHerr deleted the tropical.extratropical field. Please do that in yours/ verify that it otherwise matches, and try again. Another option could be to do a fresh download of what's there now and paste your changes to the end.

I hope I didn't make this more complicated. Currently the file is changed to have my data and the deleted tropical.extratropical field.
I ended up doing a separate commit (as opposed to a pull request) and it appears to have gone straight to the master. I did this because github desktop told me I hadn't committed anything from yesterday, but it seems to be fixed now. Can you please double check? The last entry on the csv should 17591.

That did not work.

repo: forc-db/ForC


text:
Use curl retries instead of sleeping in tests

Newer Circle CI images contain curl that supports --retry-connrefused, so it's no longer necessary to use the workaround of sleeping.
In addition, --fail has been added to the test curl calls, to ensure the test verifies that the function server response was an HTTP 200.

So for some reason, --retry-connrefused still doesn't retry this connection refused case?
+ curl http://localhost:54321 --connect-timeout 3 --retry 5 --retry-connrefused --fail ...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (7) Failed to connect to localhost port 54321: Connection refused

Exited with code exit status 7

I then tried --retry-all-errors, however the curl in the Circle CI images is not new enough to support that (even newer) option.
As such, I guess we're still out of luck for now :-/

repo: forcedotcom/sf-fx-runtime-java


text:
Press F5 to run code and get BufferExceptionTypeError then can not run the code successfully

I use the latest version. But I meet such error:
atom-python-run: BufferExceptionTypeError: Cannot read property 'then' of undefined: You failed to create the file before attempting to execute it. You must create the file first and then execute it after it has been created.
I have checked that the error comes from the function saveAndRun in atom-python-run.js file
atom.workspace.getActiveTextEditor().save().then(() => {
run({
'command': atom.config.get(atom-python-run.${key}Command),
'pause': atom.config.get(atom-python-run.${key}Pause),
});
});

if I choose an earlier version such as V0.91, it works well

Please describe steps to reproduce this error in detail. Step by step from beginning to end, how do you trigger the error?

I meet this error too.
I just open a .py file and press f5\f6 and the error message popup
atom-python-run: BufferExceptionTypeError: Cannot read property 'then' of undefined: You failed to create the file before attempting to execute it. You must create the file first and then execute it after it has been created.

For any users/developers experiencing similar issues posting a comment; please consider the following questions:

did you read the README.md?
what version of atom are you using?
what version of atom-python-run are you using?
are you using atom-python-runs default settings?
are you using atoms console to help you debug?
did the .py file previously exist? (did you save as first and then f5 after?)
does this happen with any .py files? (where files previously existed and failed to execute?)
can you execute your .py file using python or idle without errors?
do the file paths use spaces? (i.e. c:\users\user name\documents\my python\hello world.py)

if the file paths use spaces, refer to issue #60 instead, and note that issue #67 led you there

To verify what @FIRECONTER originally stated, this block causes this issue.
atom.workspace.getActiveTextEditor().save().then(() => {
    run({
        'command': atom.config.get(`atom-python-run.${key}Command`),
        'pause': atom.config.get(`atom-python-run.${key}Pause`),
    });
});

From what I can tell, this is because save().then() acts as a Promise object which expects something to be returned.
The error TypeError: Cannot read property 'then' of undefined reflects that it expects the function saveAndRun() to return something signifying that the promise has been resolved.
Because nothing is returned, the try...catch statement catches this exception. The exception is mangled by the text that is added to error using atom.notifications.addError().
For a whole picture, this is what it looks like now.
function saveAndRun(key) {
    try {
        console.log(`${key} pressed`)
        atom.workspace.getActiveTextEditor().save().then(() => {
            run({
                'command': atom.config.get(`atom-python-run.${key}Command`),
                'pause': atom.config.get(`atom-python-run.${key}Pause`),
            });
        });
    } catch(error) {
        atom.notifications.addError(`atom-python-run: \`BufferException${error}\`: You failed to create the file before attempting to execute it. You must create the file first and then execute it after it has been created.`);
        return;
    }
}

In theory, since we expect a Promise to be returned, we could just say return save.then(run()) and clean up the error message.
Because I don't know exactly what causes the issue, I can't be sure. I haven't been able to reproduce this error at all. I need in depth, step by step, information so I can reproduce it. Although, if someone figured that out, I'm sure they would be able to surmise a fix for it.
For now, It could be changed to this instead.
function saveAndRun(key) {
    try {
        console.log(`${key} pressed`);
        // save() returns a promise
        let save = atom.workspace.getActiveTextEditor().save();
        // Atom expects that promise to be resolved or rejected
        return save.then(() => {
            run({
                'command': atom.config.get(`atom-python-run.${key}Command`),
                'pause': atom.config.get(`atom-python-run.${key}Pause`),
            });
        });
    } catch(error) {
        // the promise failed and the exception was caught
        console.log(`atom-python-run: saveAndRun: ${error}`);
        atom.notifications.addError(`atom-python-run: \`${error}\`.`);
        // nothing to do, so we return undefined here
        return;
    }
}

Again, I can't guarantee that this will work. I would need someone to test this, see if it works, and then report the results here.

Look here for help on how to access and use the Console Log.
You can see here that the Console Log and cp Log are different logging mechanisms.
Feel free to express if you're confused and/or need additional clarification. If you've found a fix, please post it here, make a pull request, or create a fork with an implemented fix.

authors: FIRECONTER neederhow xovertheyearsx

repo: foreshadow/atom-python-run


text:
Fails to show issues in some projects

The problem was found with issues of
https://github.com/git-for-windows/git/issues
I can file new issue but can't view existing or even just created one. Just empty field is shown.

Maybe an loading issue?!
I have noticed that there is no loading indicator or something... Maybe the page isn't loading. Maybe wait longer?!

The issue is known and fixed #583, but the version in Google Play hasn't been updated #736

authors: Maximus5 StefMa jonan

repo: forkhubs/android


text:
feat(vue): allow useAutoAnimate with Vue component ref

This adds support for using useAutoAnimate with a component as the parent.
When you add ref="parent" to a component, the element is found at parent.value.$el.  This update checks for component elements or a plain HTML element ref.
Without this change, you have to create a proxy ref to extract the element in the onMounted hook, like this:
<script setup lang="ts">
const [autoAnimateParent, enable] = useAutoAnimate()

// create a "proxy" dragParent ref which will hold a ref to the component.
const dragParent = ref()

// assign the component's $el to the autoAnimateParent once the DOM mounts
onMounted(() => (autoAnimateParent.value = dragParent.value?.$el))
</script>

<template>
  <VueDraggable
    ref="dragParent"
    v-model="items"
    :animation="300"
  >
    <!-- ... -->
  </VueDraggable>
</template>

This PR doesn't address the types.  I'm not sure the best way to handle that.  I did pull the code into a local repo to try it out and took this screenshot of the type issue.

If you want to instruct me how to proceed on the types, I'll update the code.  Or feel free to edit this PR directly.

I just noticed this other PR.  https://github.com/formkit/auto-animate/pull/159
I think their objective is the same, based on this comment: https://github.com/formkit/auto-animate/pull/159#issuecomment-1704555164.

Sorry about being lazy with the types.  I'm not a TS ninja by any means, but I took a crack at it.  Maybe you'll see a more optimal way to handle the ElementOrComponentRef type:
import { onMounted, ref, watchEffect } from 'vue'
import type { Directive, Plugin, Ref } from 'vue'
import type {
  AnimationController,
  AutoAnimateOptions,
  AutoAnimationPlugin,
} from '@formkit/auto-animate'
import autoAnimate, { vAutoAnimate as autoAnimateDirective,
} from '@formkit/auto-animate'

export const vAutoAnimate: Directive<
  HTMLElement,
  Partial<AutoAnimateOptions>
> = autoAnimateDirective

export const autoAnimatePlugin: Plugin = {
  install(app) {
    app.directive('auto-animate', vAutoAnimate)
  },
}

export type ElementOrComponentRef<T extends Element> = T & { $el?: Element }

/**
 * AutoAnimate hook for adding dead-simple transitions and animations to Vue.
 * @param options - Auto animate options or a plugin
 * @returns A template ref. Use the `ref` attribute of your parent element
 * to store the element in this template ref.
 */
export function useAutoAnimate<T extends Element>(
  options?: Partial<AutoAnimateOptions> | AutoAnimationPlugin,
): [Ref<ElementOrComponentRef<T>>, (enabled: boolean) => void] {
  const element = ref<ElementOrComponentRef<T>>()
  let controller: AnimationController | undefined
  function setEnabled(enabled: boolean) {
    if (controller)
      enabled ? controller.enable() : controller.disable()
  }
  onMounted(() => {
    watchEffect(() => {
      const el = element.value?.$el || element.value
      if (el instanceof HTMLElement)
        controller = autoAnimate(el, options || {})
    })
  })

  return [element as Ref<T>, setEnabled]
}

repo: formkit/auto-animate


text:
How to subscribe on value changes of hidden field?

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[X] support request

Let's say I have two hidden fields as following:
          {
            className: 'col-md-4',
            key: 'selectTwo',
            type: 'select',
            templateOptions: {
              label: 'Select Two',
              placeholder: 'Select Two',
              required: true,
              options: [],
            },
            hideExpression: 'model.selectOne !== "1"',
            lifecycle: {
              onInit: (form, field) => {
                field.formControl.setValue('');
                field.templateOptions.options = this.selectService.getSelectTwoOptions();
              },
            },
          },
          {
            className: 'col-md-4',
            key: 'selectThree',
            type: 'select',
            templateOptions: {
              label: 'System Concentrator Port',
              placeholder: 'System Concentrator Port',
              required: true,
              options: [],
            },
            hideExpression: 'model.selectOne !== "1"',
            lifecycle: {
              onInit: (form, field, model) => {
                field.formControl.setValue('');
                // Here the "selectTwo" formControl does not exist yet,
                // so I cannot subscribe to its value changes, that is I cannot do:
                form.get('selectTwo').valueChanges.pipe(
                   takeUntil(this.onDestroy$),
                   startWith(form.get('selectTwo').value),
                   tap(selectTwoId => {
                     field.formControl.setValue('');
                     field.templateOptions.options = this.selectService.getSelectThreeOptions(Number(selectTwoId));
                   }),
                ).subscribe();
              },
            },

having the same hideExpression related to the value of another non hidden field (selectOne).
Now, as you can see from the previous pice of code, I would like to subscribe to valueChanges on the selectTwo hidden field, inside the selectThree hidden field.
As expected, it's not going to work inside the onInit lifecycle hook, since at that time none of the hidden fields exist yet.
I think I can subscribe inside another lifecycle hook, e.g., doCheck or afterViewChecked, but I don't know if that is the right way to go since I want to subscribe only once while those hooks are fired several times.
Another options I was thinking is to subscribe directly to the form value changes, i.e., form.valueChanges.subscribe().
Any suggestion on this?

I found the following solution:
  lifecycle: {
    onInit: (form, field) => {
      form.valueChanges.pipe(
        takeUntil(this.onDestroy$),
        filter(fields => ( fields.selectTwo
                           && ( fields.selectTwo !== '' )
                           && ( form.get('selectTwo').dirty ))),
        tap(fields => {
          // This is required to avoid the "Maximum call stack size exceeded" ERROR
          form.get('selectTwo').markAsPristine();

          field.formControl.setValue('');
          field.templateOptions.options = this. selectService.getSelectThreeOptions(Number(fields.selectTwo));
        }),
      ).subscribe();
    },
  },

So, please close this.

or subscribe to field.formControl

@aitboudad can you show me an example of subscribing to formControl ?

@ugogiordano87 try with:
const selectTwoField = field.parent.fieldGroup.find(f => f.key === 'selectTwo');
selectTwoField.formControl.valueChanges.
....

repo: formly-js/ngx-formly


text:
Add exclude list to GCPCMEKRotationConstraintV1

We need to add an exclude for the GCPCMEKRotationConstraintV1, because once created, KMS keychains cannot be removed. As long as we don't have the 'delete keychain' option in the KMS API, we should be able to exclude unused keychains.

I'm not sure what the ask here is, should this be in policy-library?

authors: briantkennedy ocervell

repo: forseti-security/config-validator


text:
High number of drops for April - Pos Rep Bot

I have been watching the performance for this bot for the month of April (Pos Rep Bot - 0xd6e19ec6dc98b13ebb5ec24742510845779d9caf439cadec9a5533f8394d435f) and it looks like it has a high number "tx_drops" for BSC chain, this issue might be also extending to the rest of the chains. I would appreciate if you can check it out, and consider sharding as a potential solution to mitigate this.

fixed with version 0.0.7

repo: forta-network/starter-kits


text:
Xcode 6.3.2 can't work.

I already add DVTPlugInCompatibilityUUID of Xcode 6.3.2.

#39 Fixes this problem.

authors: WallaceYYLi dmitriiivashko

repo: fortinmike/XcodeBoost


text:
Button Delegate Error

In InterfaceController.m there is following error:
Overlapping accesses to 'buttonDelegate', but modification requires exclusive access; consider copying to a local variable

I think this is fixed in 1.1.1, is it not?

authors: fortmarek fturobin

repo: fortmarek/SwipeViewController


text:
Code to Convert any 40x11 Translucent Gray-Scaled Bitmap to Hex Arrays

Fixes #17
Changes:

a singleton to convert drawable to hex arrays

Screenshots for the change:

A lot of magic numbers are used. Extract into constants and variables

Sure Sir

@iamareebjamal Done, Have a look :)

@iamareebjamal Fixed, Have a look.

@iamareebjamal Done, fixed all the until's

repo: fossasia/badge-magic-android


text:
widget overflow resolved

Fix #1066
Please find attached screenshots to verify

Summary by Sourcery
Bug Fixes:

Resolve widget overflow issue by adjusting layout with Expanded widgets in draw_badge_screen.dart and save_badge_dialog.dart.

Fix failing checks. Else looks good!

failing checks fixed

repo: fossasia/badgemagic-android


text:
Show user friendly error message when user tries to publish a past event

Describe the bug
Show user friendly error message when user tries to publish a past event.
Would you like to work on the issue?
Open to all.

@iamareebjamal Is this issue relevant. Can I take this one?

repo: fossasia/open-event-organizer-android


text:
gh-page rebuild failing.

Actual Behaviour
gh-page rebuild is failing.
Expected Behaviour
All webgen assosciated websites should reflect new changes.
Would you like to work on the issue?
Yes, but would need write access to this project.@mariobehling please check this.

@agbilotia1998 You can send PR to the gh-pages branch manually. We can try it & see if the auto deploy on gh-pages start working again.

@geekyd @mariobehling  I would be needing gh-token of this repository to test if the problem is something like travis-ci timeout or something else.

The issue is still there.

@mariobehling @geekyd @ParthS007 please check this screenshot

@agbilotia1998  I think there is problem with token , will check this :+1:

repo: fossasia/open-event-webapp


text:
Filters for mobile view

Short description of what this resolves:
Provides filter facility based on tracks and rooms as a drop-down facility in mobile view.
Changes proposed in this pull request:
When the viewport size is not enough to display filters on the right,
they get replaced by a drop-down widget to display filters.
Following is a small gif showing the changes made.

Fixes #1781

Here is the demo link
FOSSASIA-Summit#1793

@agbilotia1998, @ParthS007 please review

A deployment link has already been provided above.
However, here it is again
https://rohan598.github.io/FOSSASIA-Summit-1793/

@ParthS007 please review

Thanks!
Please resolve conflicts and check the following:

Left align filters
instead of opening several submenus expand room or track filter in the same level if possible. Hopefully this ensure better usability in mobile view.

About the submenus, if I will expand them in the same level the menu becomes too long and has a blocking presence even after the scroll option. But if you insist,I’ll do that and send a demo link for the same.

+1 to @joeytwiddle 's review.

Thanks @joeytwiddle for your suggestion, I had missed it somehow. I have incorporated your suggested changes into my pull request.

Please review @agbilotia1998  @ParthS007 @joeytwiddle @mariobehling,
I have made all the necessary changes
Demo link:
https://rohan598.github.io/FOSSASIA-Summit-1793/
Demo

I apologize for the delay as I was busy with my college work.

Please review @joeytwiddle I have made the suggested changes.

Please resolve conflicts.

@joeytwiddle Would be great, if you finish off this PR.

Closing this as #1860 is opened for the same issue.


text:
WIP Alternative breaking PR for versionstamp packing

This one takes a more opinionated approach to packing, to ensure that packing of values with incomplete versionstamp will append the versionstamp offset at the end.
To further expand on this, we can expose an enum with Complete(bytes) and Incomplete(bytes) to force the use with set_versionstampedX

May I close this?

authors: PierreZ behos

repo: foundationdb-rs/foundationdb-rs


text:
Is it possible to use stdstore to write to an internal mapping?

I tried this but it didn't work:
stdstore.target(addr).sig("_mappingName(address)").with_key(token).checked_write(value);

I got the following error:

FAIL. Reason: stdStorage find(StdStorage): No storage use detected for target.

The sig must be a public getter. It works by calling the getter, recording all slots accessed during that read, then looping through each read slot, modifying it, calling the getter again to see if we found the correct slot
Some discussion on an alternative approach to leverage storage layout instead in https://github.com/foundry-rs/foundry/issues/4022 and in the comment linked there

Thank you, @mds1.

repo: foundry-rs/forge-std


text:
The position of chisel command line prompt on windows is displayed incorrectly

Component
Chisel
Have you ensured that all of these are up to date?

[ ] Foundry
[ ] Foundryup

What version of Foundry are you on?
forge 0.2.0 (1d9a34e 2023-03-07T00:06:14.4107988Z)
What command(s) is the bug in?
No response
Operating System
Windows
Describe the bug
The position of the cursor is wrong

I analyzed the code, the problem lies in the dispatcher.get_prompt function
This can be modified chisel.rs main function
let prompt = if cfg!(windows) { format!(">") }else {dispatcher.get_prompt()};

nice, catch, do you want to submit a PR with a fix?
unable to test this myself because I don't have access to a windows machine.

nice, catch, do you want to submit a PR with a fix? unable to test this myself because I don't have access to a windows machine.

Ok. Please wait for a moment

unable to test this myself because I don't have access to a windows machine.

🫡
https://www.virtualbox.org/

authors: DaniPopes daog1 mattsse

repo: foundry-rs/foundry


text:
feature request: add --token option to cast balance

Component
Cast
Describe the feature you would like
Currently I use following ways to get erc20 / erc721 balance.
cast call [token_address] "balanceOf(address) returns uint256" --rpc-url=https://rpc.ankr.com/eth

Every time I spend some time to write the method sig params.
IMHO, as erc20 / erc721 become standard and become some common sense rules, I want to make a short cut to get erc20 / erc721 balance.
Like:
cast balance 0x162c6270266667ccf5a9ed752b5d6a2bdc0f90de --token=0x5a98fcbea516cf06857215779fd812ca3bef1b32  --rpc-url=https://rpc.ankr.com/eth

And I would like to implement it.
Additional context
No response

@fenghaojiang feel free to go for it!

The proposed UX of cast balance --token <address> lgtm!

supportive

authors: Evalir fenghaojiang gakonst mds1


text:
Revert "feat(cheatcodes): Make expectCall only work for the next call's subcalls"

Reverts foundry-rs/foundry#4986
this was blocked for 1.0 and should not have been merged yet.

Could you please cut another release, @mattsse?
Our tests are failing now:
https://app.warp.dev/block/mLxbng37Ztgs9OxSko3PMf

already running, sorry for the inconvenience

TY @mattsse!
Also, FWIW, I don't think the implementation in #4986 works as expected.
Our tests already adhere to the rule of not counting calls made at the root test level, so it's strange that we started getting errors.
FWIW, the depth in our case is greater than usual, as we're routing the calls via delegatecall, specifically via the execute function in PRBProxy:
https://github.com/PaulRBerg/prb-proxy/blob/b5338c7e5a3a10468f584ea1f1caca08a510939a/src/PRBProxy.sol#L87
Lmk if you'd like access to the private repo @mattsse @Evalir for investigation. We also have a CI run that failed:
https://github.com/sablier-labs/v2-periphery/actions/runs/5071179085/jobs/9107504005

Hey @PaulRBerg, gotcha, this is great feedback. We're gonna start merging these into a feature branch, so we can run extended QA on big repos that use foundry heavily—we'll integrate prb-math and prb-proxy into external integration tests for this as well.
Would love access to the private repo—that way I can run QA and debug what might not be working as expected!

Can confirm that the latest release (3764b55 2023-05-24T17:24:24.100981000Z) works now, thanks guys.

Amazing, great to hear. Will probably be poking around really soon and testing with the new/upcoming changes!

authors: Evalir PaulRBerg mattsse


text:
fix: overflow in randomUint

Don't overflow max - min by swapping them before. CC @yash-atreya @mds1
The other alternative is to outright reject max > min

Actually I will just make it reject max > min


text:
Armor Upgrade Item Summary & Tooltip Fix

Fixes issue #1052, where the upgrade chat data was being gathered incorrectly from upgrade items. Also fixes an issue where the properties tooltip was displaying "undefined" for properties without one.

Tooltip bug this fixes shown below.

My concern with this is with how it relates to existing upgrades; some have armorType set, others allowedArmorType and others both. It seems allowedArmorType is the one more commonly used

What's the resolution for that? Go through and edit all the upgrades in the compendium to use allowedArmorType?

As far as I can tell, on any item with both armorType and allowedArmorType, allowedArmorType is correct. If an item has only armorType, then that is correct. Therefore I think the way forward is to simply delete armorType on items with both, and migrate armorType to allowedArmorType on items with only armorType.

As far as I can tell, on any item with both armorType and allowedArmorType, allowedArmorType is correct. If an item has only armorType, then that is correct. Therefore I think the way forward is to simply delete armorType on items with both, and migrate armorType to allowedArmorType on items with only armorType. Migrate in the most basic definition of the word; just needs to be a simple compendium node script.

I've added a node script for this in https://github.com/foundryvtt-starfinder/foundryvtt-starfinder/pull/1182/commits/8efad9426ddaf9eeb8799570528f2d26bf10431a and updated the relevant compendium items.


text:
Rolldata Display is not sensibly rounded

I've got a monster that does less damage based on it's current HP.
The damage formula is (@attributes.hp.pct/100) * 10d10.
The current HP of the monster in this example is 57 of a 100HP maximum.
The display on the sheet is:

The sheet display should avoid precision errors or intentionally round as some HP values can actually be quite long decimals.
e.g. 10/15 would be a recurring decimal

Seems like a much easier fix on your end, given how division beyond 2 and 4 just doesn't exist in the normal rules. Wrap the @ value in round() then do your division.

Rounding to two decimals would probably save a few minor headaches.

authors: JPMeehan krbz999 revilowaldow


text:
Question about NextMessageAsync

Is there a requirement to call this method from within a Command or within the Command class?
I wrote a small Utilities class in which I implemented the method NextMessageByUserContains(string userID, string message). This is meant to be used for finding replies to a command.
Whenever I call this method from my Command class it results in NullReferences. Then when I move the method over to my Command class it works fine. Is there something I am missing?

i also get null as response

This still persists, but I have gotten around it by just doing all the work in my helper utility class and returning the value and using the method in the command module. It does not work in my utility class.

license: ISC


text:
Fixes Image Subfolder Date Formats

The subfolders under images/ for each post were named in a manner that was inconsistent with the post filenames (and that also caused them to be ordered incorrectly).  This corrects the naming of those folders to match their respective post filenames in _posts/.
I know this can be a contentious topic, and I'm happy to discuss this, but this change seemed reasonable to me.  Previously, the image subfolders would order by month, then day, then year, which is not an ordering that makes sense in any context I'm aware of.  This and the inconsistency between post filenames and image folder names bugged me.

This is good. :shipit:

authors: FGS-PKnouff fgs-pwright