Guide on running these versions with apache (mod php)

Any idea on how we could use these old php version (5.5) on apache

Hi !
AFAIK, we don't maintain 5.5, we start at 5.6.

Hi !
AFAIK, we don't maintain 5.5, we start at 5.6.

Ok, 5.6 is also fine for us.

First you need to add the overlay of this flake in your own configuration.
In the whole process, this is probably the most complex part. An overlay is just the default standard mechanism to extends pkgs in Nix by adding a layer on top of your configuration so that when you do pkgs.php56, it works.
Then, once it's done, a quick search on Google led me to https://nixos.wiki/wiki/PHP, where I can see a snippet related to Apache.
You would just need to replace pkgs.php with pkgs.php56 and it should be good to go.

Technically it should work, but using raw /nix/store/... paths is a bad practice since it can be garbage collected.
I would simply recommend you to add it to your profile using nix profile so now your can be sure that it will never be GC'd.
❯ nix profile install github:fossar/nix-phps#php56
❯ php -v
PHP 5.6.40 (cli) (built: Jan  9 2019 10:25:59)
Copyright (c) 1997-2016 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2016 Zend Technologies
   with Zend OPcache v7.0.6-dev, Copyright (c) 1999-2016, by Zend Technologies
❯ type php
php is /home/pol/.nix-profile/bin/php
❯

If there are no other php on your system and if the user running apache is the same user who run nix profile, yes.

Seems like mod-php embeds php within apache, and doesn't call cli process at all, so this idea wouldn't work

In theory, it should be possible but unless Apache has stable ABI for modules, you would probably also need to ensure that the PHP is linked against the same version of Apache as the one you are running (and possibly other parameters).
I think at that point it would be easier to replace the Ubuntu’s Apache with a one from Nix.

In theory, it should be possible but unless Apache has stable ABI for modules, you would probably also need to ensure that the PHP is linked against the same version of Apache as the one you are running (and possibly other parameters).
I think at that point it would be easier to replace the Ubuntu’s Apache with a one from Nixpkgs.

Yes but does nixpkgs apache have older versions of mod-php?

No. But that is what this repo is for.

No. But that is what this repo is for.

Ok, so older modphp versions can be added to this flake?

If you want to to run Apache from Nix, you are best off looking at the relevant NixOS module. For example, this is how you get PHP with mod_php:
https://github.com/NixOS/nixpkgs/blob/8a3e1cf40a6eaeb122c8321b97a0518cfa6ed779/nixos/modules/services/web-servers/apache-httpd/default.nix#L21

If you want to to run Apache from Nix, you are best off looking at the relevant NixOS module. For example, this is how you get PHP with mod_php:
https://github.com/NixOS/nixpkgs/blob/8a3e1cf40a6eaeb122c8321b97a0518cfa6ed779/nixos/modules/services/web-servers/apache-httpd/default.nix#L21

Yes, but can i replace the php version in them to older ones like 5.6

Sure, just do as mentioned above https://github.com/fossar/nix-phps/issues/325#issuecomment-1934387999

Sure, just do as mentioned above #325 (comment)

I'm yet to try this overlays approach, it would be handier if it was prepackaged in this flake itself, basically I'm imagining this flake to be a simple way to maintain legacy versions of our software, we are also exploring lxd containers in this space.

You do not need the overlays approach but it the simplest way to use this flake. Or at least with fewest caveats – you could just use the packages.${system}.php output of the flake directly, but then you would need to make sure the Nixpkgs revision is the same as the one used to built PHP in this flake.
We are not going to include Apache in this flake, that just does not compose.

i don't think what has been discussed in this issue is really the biggest challenge with what @aszenz is attempting to do - generating a configuration file for apache and running the software is going to be the biggest challenge when attempting to run a nix provided apache+php on ubuntu, in my opinion

so how can @aszenz proceed? personally i would suggest using system-manager as it would make running older php versions with apache on ubuntu relatively easy compared to other options
i was able to get the latest version of apache running php version 5.6 on ubuntu in just a few minutes from scratch with the following configuration:
{ nixosModulesPath, config, pkgs, lib, ... }:
{
  imports = [
    "${nixosModulesPath}/services/web-servers/apache-httpd"
    ./compat.nix
  ];

  nixpkgs.hostPlatform = "x86_64-linux";

  services.httpd = {
    enable = true;
    enablePHP = true;
    user = "www-data";
    group = "www-data";
    phpPackage = pkgs.php56;

    virtualHosts."localhost" = {
      documentRoot = "/srv/www";
    };
  };
}

@aszenz if you're interested in exploring this option further i would be happy to provide additional details for you

i don't think what has been discussed in this issue is really the biggest challenge with what @aszenz is attempting to do - generating a configuration file for apache and running the software is going to be the biggest challenge when attempting to run a nix provided apache+php on ubuntu, in my opinion
so how can @aszenz proceed? personally i would suggest using system-manager as it would make running older php versions with apache on ubuntu relatively easy compared to other options
i was able to get the latest version of apache running php version 5.6 on ubuntu in just a few minutes from scratch with the following configuration:
{ nixosModulesPath, config, pkgs, lib, ... }:
{
  imports = [
    "${nixosModulesPath}/services/web-servers/apache-httpd"
    ./compat.nix
  ];

  nixpkgs.hostPlatform = "x86_64-linux";

  services.httpd = {
    enable = true;
    enablePHP = true;
    user = "www-data";
    group = "www-data";
    phpPackage = pkgs.php56;

    virtualHosts."localhost" = {
      documentRoot = "/srv/www";
    };
  };
}

@aszenz if you're interested in exploring this option further i would be happy to provide additional details for you

Thanks, i tried this on my own nixos system (the server will be ubuntu though) and got this error:
getting status of '/nix/store/jp2kw74bigwv1npgi02n22hqm7dj7r6f-source/compat.nix': No such file or directory

i don't think what has been discussed in this issue is really the biggest challenge with what @aszenz is attempting to do - generating a configuration file for apache and running the software is going to be the biggest challenge when attempting to run a nix provided apache+php on ubuntu, in my opinion
so how can @aszenz proceed? personally i would suggest using system-manager as it would make running older php versions with apache on ubuntu relatively easy compared to other options
i was able to get the latest version of apache running php version 5.6 on ubuntu in just a few minutes from scratch with the following configuration:
{ nixosModulesPath, config, pkgs, lib, ... }:
{
  imports = [
    "${nixosModulesPath}/services/web-servers/apache-httpd"
    ./compat.nix
  ];

  nixpkgs.hostPlatform = "x86_64-linux";

  services.httpd = {
    enable = true;
    enablePHP = true;
    user = "www-data";
    group = "www-data";
    phpPackage = pkgs.php56;

    virtualHosts."localhost" = {
      documentRoot = "/srv/www";
    };
  };
}

@aszenz if you're interested in exploring this option further i would be happy to provide additional details for you

Thanks i created https://github.com/numtide/system-manager/issues/72, initially tested it on NixOS before i can test it on Ubuntu server

as mentioned - if you're interested in exploring this option further i would be happy to provide additional details for you
i did not include all details so please close that ticket against system-manager - there is no problem there, the problem is that you don't have all the source
i pushed the source here so you can take a look and have a fully running example


Remove duplicated messages in the message catalog

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
I deduplicate the same definitions, "find": "Find in workspace" in the message catalog.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage: 68.577% (-0.03%) from 68.608% when pulling 37057d1da25c88515784bb7992352865ce4fb1bc on kazuhitoyokoi:dev-dedup into 196a9ae43abd4211195f4698912a758bb904e6ab on node-red:dev.


CanvasItemMaterials and ShaderParams

Hello,
I might do things bad.
I observed the cubio demo and I wanted to test things on my own using CanvasItemMaterials on Sprite.
The CanvasItemMaterials have Shader Params, on my own project (using v1.1 of course) I see that the API allows me to create ones but I can't add shader params via the editor, which leads to error when trying to script shaders.

To me this sounds more like a question, not an issue.
In order for shader params to show in the inspector, you must add some uniforms to the shader graph/script.

Thanks a lot :)


Server Setup and Web Hosting

Setup and configuration of the TDT on an internal server.

[ ] Web portal dockerization
[ ] System deployment Docker compose
[ ] Package nanobot into web portal


we have to ignore RUSTC_WORKSPACE_WRAPPER as well

This matches what we do with RUSTC_WRAPPER.

@bors r+

:pushpin: Commit f6b01440e78489c35b5478c6c72cfa6d7e52d242 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit f6b01440e78489c35b5478c6c72cfa6d7e52d242 with merge 2f2e1290777e273b2783daa7c602df33d058b363...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 2f2e1290777e273b2783daa7c602df33d058b363 to master...


Updating mention stripping scenario for Teams with special character support

Fixing bug: #3106

Pull Request Test Coverage Report for Build 94572

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 74.14%

Totals

Change from base Build 94519:
0.0%

Covered Lines:
14556

Relevant Lines:
19633

💛  - Coveralls


Update package.json

After installing NPM, run the npm run dev, the following error
module.js:440
    throw err;
    ^

Error: Cannot find module 'memory-fs'
    at Function.Module._resolveFilename (module.js:438:15)
    at Function.Module._load (module.js:386:25)
    ...

dependency write missing, and now fix it.

There's already a pull request for this: #7

Looks like #7 was closed. npm@3 is the version of npm to use with this project. I'm guessing @chaoren1641 is using npm@2, which is why they're getting the error.

Well, it should be like this. I installed a number of local nodejs version, I can not remember which version of the time.


using switch - case instead of if else

Thanks 👍

Does the new code work properly?

Ill check it, wait

Fixed - https://github.com/xHyroM/cf-workers-uploader/commit/bf04c11f8e0eccf71b117bd122b2c0dce60070e6


GEOMESA-788 Add Proper Partitioning Control To GeoSpark

This is a messy mid-term solution to the over-partitioning issue. It does not resolve the under-partitioning issue, which would require finagling the actual ranges. I didn't feel comfortable enough with the deadlines or the codebases to pursue that, but we do need to discuss it.
Coding style's a little rough, but I've tried to be mostly conformant. The added API function is deprecated, as it will be merged into the main API function down the road, once we solve the under-partitioning problem.

@elahrvivaz I removed the.... uhh... extraneous logging.

@elahrvivaz fixed the lines. check it? I'd like to get this merged in.

@jnh5y are you good with it now?

@JakeKurzer sorry, I've been out...

@elahrvivaz tomorrow I'm going to code up some handling for extremely large ranges, so that'll probably make the divergence more justified. Let's plan to talk prior to symposium tomorrow?


refactor: new usage useDisclosure return

Describe your changes
Like the usage of useQuery (https://github.com/BearStudio/start-ui-web/pull/342), I think it's a better idea to not destructure ChakraUI useDisclosure return value, because it's more readable and that simplify disclosure definition and refactor.
For example :
Not readable or complex :
const { isOpen, onOpen, onClose } = useDisclosure();
const { isOpen: isOpenConfirmModal, onOpen: onOpenConfirmModal, onClose: onCloseConfirmModal } = useDisclosure();

...

<Button onClick={onOpen}>Open</Button>
<Button onClick={onOpenConfirmModal}>Open</Button>

Easier and more readable :
const confirmModal = useDisclosure();

...

<Button onClick={confirmModal.onOpen}>Open</Button>

Checklist

[x] I performed a self review of my code
[x] I ensured that everything is written in English
[x] I tested the feature or fix on my local environment
[x] I ran the yarn storybook command and everything is working

1 code smell to check https://sonarcloud.io/project/issues?id=BearStudio_start-ui&pullRequest=346&resolved=false&types=CODE_SMELL


feat: using detox for e2e tests in /mobile

Description

to #560

Type of change

New feature (non-breaking change which adds functionality)

Closing this as it's not relevant anymore


Update autofishbot.py

Sanitized duplicator emoji

Thank you so much for the guidance!

Thank you @yudhistiraindyka !


S2E13 AMC Meeting Announcement

AMC Meeting Announcement

S2E13
Session 2 Episodes 13

Date
7-9 pm, Sunday

Location
821, 8th floor, BUPT Research Building

Chairman
Guang Chen @cgpeter96

Vice Chairman
Bo Lin @linbo0518

Previous Meeting

Minutes
Link

Blog
S2E12 (Session 2 Episodes 12)

Issue
#117

Agenda
1. Oral

Oral should be prepared for 15 - 30 minutes, and no more than 45 minutes, including Q/A.

Presenter
Paper

Zhengyuan ZHU @824zzy
面相深度学习系统的突变软件测试

2. Poster

Posters should be prepared for 5 - 10 minutes, and no more than 15 minutes, including Q/A.

Presenter
Paper

Huan LI @huan
Generating Long Sequences with Sparse Transformers, Rewon Child, et. al., 2019

Zhenggang Hou @houzhenggangGit
Convolutional Pose Machines

Yun Liu @fuyunfuyun666
Detecting Curve Text in the Wild: New Dataset and New Solution

Jiaming Duan @vdeamov
TADAM: Task dependent adaptive metric for improved few-shot learnin, Oreshkin, et. al., 2018

Dixi Chen @chen-dixi
Almost Unsupervised Text to Speech and Automatic Speech Recognition, Yi Ren, et. al., ICML2019

3. After Party
Everyone will be welcome. You can join or not join as you want (not required by the AMC), and change your mind at anytime.

After Party will be started right after all the meeting agenda had been fulfilled.
We will find a bar and drink beers and talk whatever we want. (we can expect that it will be related to meeting topics of the day)
The bill will be paid AA

Huan LI
Zhengyuan ZHU
ANY ONE WILL BE WELCOME

New Comers

Referrer
Newcomer
GitHub
Bio

Jiaming Duan @vdeamov
张楚镕
@zhangchurong
OCR

Mizou, CEO and founder of TrappyAI, a startup creates chatbots with cool personalities.  @huan

@cgpeter96 Is oral paper's name correct?


adding composer json file

Added a composer.json file to this project so that it can be used via packagist.org directly. Let me know if I got any of your info wrong.

Thank you :) :+1:


[CI] Run uss_qualifier tests in parallel jobs

This splits in parallel jobs the execution of the different USS qualifier configurations.
This allows the CI execution time to be roughly halved.
Do note that I have not tested that the push to GH pages is correct, since it is enabled only on main.

does this cause, e.g., like 10 DSS instances to be spun up?

Yes exactly, it will spawn up separate VMs on which the full stack will be deployed.

As long as this isn't going to cause us to hit quota limits 10x faster than before, then this seems great.

That I'm not sure, I did not consider this aspect at all TBH.
How are the quotas computed currently? This setup does consume more network bandwidth (since everything is downloaded 10X times for each run), the total minutes of CPU usage will be higher (because of the setup overhead on each VM).
Considering those aspects, should I merge this PR or not?

And also just to capture the outcome from the discussion in weekly meeting: I believe our decision was to go ahead and merge this PR (when ready) now.  We can roll back if we see any problems, and we can also see if things can be more optimized after merging (e.g., build image once and reuse in multiple jobs).  But, we didn't identify any specific reason not to speed up tests in this way.

And also just to capture the outcome from the discussion in weekly meeting: I believe our decision was to go ahead and merge this PR (when ready) now. We can roll back if we see any problems, and we can also see if things can be more optimized after merging (e.g., build image once and reuse in multiple jobs). But, we didn't identify any specific reason not to speed up tests in this way.

My bad I went a bit fast here. I'm resetting it and merging it.


🐛 [cloud_functions] Error/warning after upgrade to Flutter 2.5.0 , The plugin cloud_functions uses a deprecated version of the Android embedding.

Bug report
I have this error/warning after upgrading to Flutter 2.5.0

The plugin cloud_functions uses a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to see if this plugin supports the Android V2 embedding. Otherwise, consider removing it since a future release of Flutter will remove these deprecated APIs.
If you are plugin author, take a look at the docs for migrating the plugin to the V2 embedding: https://flutter.dev/go/android-plugin-migration.

Steps to reproduce
I have this error after upgrading to Flutter 2.5.0
Expected behavior
I have no warning/error again
Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.0, on macOS 11.2.1 20D74 darwin-x64, locale en-ID)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] VS Code (version 1.60.0)
[✓] Connected device (2 available)

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.14.0
Flutter SDK 2.5.0
main_app 1.0.0+1

dependencies:
- add_2_calendar 2.1.0 [flutter]
- after_layout 1.1.0 [flutter]
- cached_network_image 3.1.0 [flutter flutter_cache_manager octo_image cached_network_image_platform_interface cached_network_image_web]
- cloud_firestore 2.5.1 [cloud_firestore_platform_interface cloud_firestore_web collection firebase_core firebase_core_platform_interface flutter meta]
- cloud_functions 3.0.2 [cloud_functions_platform_interface cloud_functions_web firebase_core firebase_core_platform_interface flutter]
- connectivity 3.0.6 [flutter meta connectivity_platform_interface connectivity_macos connectivity_for_web]
- cupertino_icons 1.0.3
- device_info 2.0.2 [flutter device_info_platform_interface]
- firebase_analytics 8.3.1 [firebase_analytics_platform_interface firebase_analytics_web firebase_core flutter meta]
- firebase_auth 3.1.0 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 1.6.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_crashlytics 2.2.1 [firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface flutter stack_trace]
- firebase_messaging 10.0.6 [firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web flutter meta]
- firebase_performance 0.7.0+9 [firebase_core firebase_core_platform_interface firebase_performance_platform_interface flutter]
- firebase_remote_config 0.10.0+5 [firebase_core firebase_core_platform_interface firebase_remote_config_platform_interface flutter]
- firebase_storage 10.0.3 [firebase_core firebase_core_platform_interface firebase_storage_platform_interface firebase_storage_web flutter]
- flash 2.0.1 [flutter]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_image_compress 1.0.0 [flutter]
- flutter_linkify 5.0.2 [flutter linkify]
- flutter_local_notifications 7.0.0 [flutter flutter_local_notifications_platform_interface platform timezone]
- flutter_localizations 0.0.0 [flutter intl characters clock collection meta path typed_data vector_math]
- flutter_riverpod 0.14.0+3 [collection flutter meta riverpod state_notifier]
- font_awesome_flutter 9.1.0 [flutter]
- geocoding 2.0.0 [flutter geocoding_platform_interface]
- geolocator 7.3.1 [flutter geolocator_platform_interface geolocator_web]
- google_maps_flutter 2.0.6 [flutter flutter_plugin_android_lifecycle google_maps_flutter_platform_interface]
- google_sign_in 5.0.5 [flutter google_sign_in_platform_interface google_sign_in_web meta]
- hive 2.0.4 [meta crypto]
- hive_flutter 1.1.0 [flutter hive path_provider path]
- http 0.13.3 [async http_parser meta path pedantic]
- image 3.0.2 [archive meta xml]
- image_cropper 1.4.1 [flutter]
- image_picker 0.8.2 [flutter flutter_plugin_android_lifecycle image_picker_for_web image_picker_platform_interface]
- intl 0.17.0 [clock path]
- introduction_screen 2.1.0 [flutter dots_indicator]
- ios_utsname_ext 2.0.0 [flutter]
- meta 1.7.0
- mime 1.0.0
- mockito 5.0.15 [analyzer build code_builder collection dart_style matcher meta path source_gen test_api]
- package_info 2.0.2 [flutter]
- path_provider 2.0.2 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- remove_emoji 0.0.6
- share_plus 2.1.4 [meta mime flutter share_plus_platform_interface share_plus_linux share_plus_macos share_plus_windows share_plus_web]
- shared_preferences 2.0.6 [meta flutter shared_preferences_platform_interface shared_preferences_linux shared_preferences_macos shared_preferences_web shared_preferences_windows]
- shimmer 2.0.0 [flutter]
- syncfusion_flutter_datepicker 19.2.49 [flutter intl syncfusion_flutter_core]
- url_launcher 6.0.9 [flutter meta url_launcher_linux url_launcher_macos url_launcher_platform_interface url_launcher_web url_launcher_windows]
- uuid 3.0.4 [crypto]
- validators 3.0.0
- webview_flutter 2.0.10 [flutter]

dev dependencies:
- build_runner 2.1.2 [args async analyzer build build_config build_daemon build_resolvers build_runner_core code_builder collection crypto dart_style frontend_server_client glob graphs http_multi_server io js logging meta mime package_config path pedantic pool pub_semver pubspec_parse shelf shelf_web_socket stack_trace stream_transform timing watcher web_socket_channel yaml]
- flutter_native_splash 1.2.0 [args image meta path xml yaml universal_io]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]
- hive_generator 1.1.0 [build source_gen hive analyzer source_helper]

transitive dependencies:
- _fe_analyzer_shared 22.0.0 [meta]
- analyzer 1.7.2 [_fe_analyzer_shared cli_util collection convert crypto glob meta package_config path pub_semver source_span watcher yaml pedantic]
- archive 3.1.2 [crypto path]
- args 2.2.0
- async 2.8.1 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- build 2.1.0 [analyzer async convert crypto glob logging meta path]
- build_config 1.0.0 [checked_yaml json_annotation path pubspec_parse yaml]
- build_daemon 3.0.0 [built_collection built_value http_multi_server logging pedantic path pool shelf shelf_web_socket stream_transform watcher web_socket_channel]
- build_resolvers 2.0.4 [analyzer async build crypto graphs logging path package_config pool pub_semver stream_transform yaml]
- build_runner_core 7.1.0 [async build build_config build_resolvers collection convert crypto glob graphs json_annotation logging meta path package_config pedantic pool timing watcher yaml]
- built_collection 5.1.0
- built_value 8.1.1 [built_collection collection fixnum meta]
- cached_network_image_platform_interface 1.0.0 [flutter flutter_cache_manager]
- cached_network_image_web 1.0.0 [flutter flutter_cache_manager cached_network_image_platform_interface]
- characters 1.1.0
- charcode 1.3.1
- checked_yaml 2.0.1 [json_annotation source_span yaml]
- cli_util 0.3.3 [meta path]
- clock 1.1.0
- cloud_firestore_platform_interface 5.4.1 [collection firebase_core flutter meta plugin_platform_interface]
- cloud_firestore_web 2.4.1 [cloud_firestore_platform_interface collection firebase_core firebase_core_web flutter flutter_web_plugins js]
- cloud_functions_platform_interface 5.0.10 [firebase_core flutter meta plugin_platform_interface]
- cloud_functions_web 4.0.12 [cloud_functions_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- code_builder 4.1.0 [built_collection built_value collection matcher meta]
- collection 1.15.0
- connectivity_for_web 0.4.0 [connectivity_platform_interface flutter_web_plugins flutter]
- connectivity_macos 0.2.0 [flutter]
- connectivity_platform_interface 2.0.1 [flutter meta plugin_platform_interface]
- convert 3.0.1 [typed_data]
- cross_file 0.3.1+4 [flutter meta]
- crypto 3.0.1 [collection typed_data]
- dart_style 2.0.3 [analyzer args path pub_semver source_span]
- device_info_platform_interface 2.0.1 [flutter meta plugin_platform_interface]
- dots_indicator 2.0.0 [flutter]
- fake_async 1.2.0 [clock collection]
- ffi 1.1.2
- file 6.1.2 [meta path]
- firebase 9.0.1 [http http_parser js]
- firebase_analytics_platform_interface 2.0.1 [flutter meta]
- firebase_analytics_web 0.3.0+1 [firebase firebase_analytics_platform_interface flutter flutter_web_plugins meta]
- firebase_auth_platform_interface 6.1.0 [firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 3.1.0 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core_platform_interface 4.0.1 [collection flutter meta plugin_platform_interface]
- firebase_core_web 1.1.0 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_crashlytics_platform_interface 3.1.2 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_platform_interface 3.0.5 [firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_web 2.0.5 [firebase_core firebase_core_web firebase_messaging_platform_interface flutter flutter_web_plugins js meta]
- firebase_performance_platform_interface 0.0.1+2 [firebase_core flutter plugin_platform_interface]
- firebase_remote_config_platform_interface 0.3.0+5 [firebase_core flutter meta plugin_platform_interface]
- firebase_storage_platform_interface 4.0.2 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_storage_web 3.0.2 [async firebase_core firebase_core_web firebase_storage_platform_interface flutter flutter_web_plugins http js meta]
- fixnum 1.0.0
- flutter_blurhash 0.6.0 [flutter meta pedantic]
- flutter_cache_manager 3.1.2 [clock collection file flutter http path path_provider pedantic rxdart sqflite uuid]
- flutter_local_notifications_platform_interface 4.0.1 [flutter plugin_platform_interface]
- flutter_plugin_android_lifecycle 2.0.2 [flutter]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- freezed_annotation 0.14.2 [collection json_annotation meta]
- frontend_server_client 2.1.0 [async path]
- geocoding_platform_interface 2.0.0 [flutter meta plugin_platform_interface]
- geolocator_platform_interface 2.3.2 [flutter plugin_platform_interface vector_math meta]
- geolocator_web 2.0.6 [flutter flutter_web_plugins geolocator_platform_interface]
- glob 2.0.1 [async collection file path pedantic string_scanner]
- google_maps_flutter_platform_interface 2.1.0 [collection flutter meta plugin_platform_interface stream_transform]
- google_sign_in_platform_interface 2.0.1 [flutter meta quiver]
- google_sign_in_web 0.10.0+1 [flutter flutter_web_plugins google_sign_in_platform_interface js meta]
- graphs 2.0.0
- http_multi_server 3.0.1 [async]
- http_parser 4.0.0 [charcode collection source_span string_scanner typed_data]
- image_picker_for_web 2.1.2 [flutter flutter_web_plugins image_picker_platform_interface meta]
- image_picker_platform_interface 2.2.0 [flutter http meta plugin_platform_interface cross_file]
- io 1.0.3 [meta path string_scanner]
- js 0.6.3
- json_annotation 4.0.1
- linkify 4.0.0
- logging 1.0.1
- matcher 0.12.10 [stack_trace]
- octo_image 1.0.0+1 [flutter flutter_blurhash]
- package_config 2.0.0 [path]
- path 1.8.0
- path_provider_linux 2.0.2 [flutter path path_provider_platform_interface xdg_directories]
- path_provider_macos 2.0.2 [flutter]
- path_provider_platform_interface 2.0.1 [flutter meta platform plugin_platform_interface]
- path_provider_windows 2.0.3 [ffi flutter meta path path_provider_platform_interface win32]
- pedantic 1.11.1
- petitparser 4.1.0 [meta]
- platform 3.0.0
- plugin_platform_interface 2.0.1 [meta]
- pool 1.5.0 [async stack_trace]
- process 4.2.3 [file path platform]
- pub_semver 2.0.0 [collection]
- pubspec_parse 1.0.0 [checked_yaml collection json_annotation pub_semver yaml]
- quiver 3.0.1 [matcher]
- riverpod 0.14.0+3 [collection freezed_annotation meta state_notifier]
- rxdart 0.27.1
- share_plus_linux 2.0.3 [share_plus_platform_interface file flutter meta url_launcher]
- share_plus_macos 2.0.2 [share_plus_platform_interface flutter]
- share_plus_platform_interface 2.0.1 [flutter meta mime plugin_platform_interface]
- share_plus_web 2.0.4 [share_plus_platform_interface url_launcher flutter flutter_web_plugins meta]
- share_plus_windows 2.0.3 [share_plus_platform_interface flutter meta url_launcher]
- shared_preferences_linux 2.0.2 [file meta flutter path path_provider_linux shared_preferences_platform_interface]
- shared_preferences_macos 2.0.2 [flutter shared_preferences_platform_interface]
- shared_preferences_platform_interface 2.0.0 [flutter]
- shared_preferences_web 2.0.1 [flutter flutter_web_plugins meta shared_preferences_platform_interface]
- shared_preferences_windows 2.0.2 [flutter file meta path path_provider_platform_interface path_provider_windows shared_preferences_platform_interface]
- shelf 1.2.0 [async collection http_parser path stack_trace stream_channel]
- shelf_web_socket 1.0.1 [shelf stream_channel web_socket_channel]
- sky_engine 0.0.99
- source_gen 1.0.3 [analyzer async build dart_style glob meta path pedantic source_span yaml]
- source_helper 1.2.0 [analyzer collection source_gen]
- source_span 1.8.1 [collection path term_glyph]
- sqflite 2.0.0+3 [flutter sqflite_common path]
- sqflite_common 2.0.0+2 [synchronized path meta]
- stack_trace 1.10.0 [path]
- state_notifier 0.7.0 [meta]
- stream_channel 2.1.0 [async]
- stream_transform 2.0.0
- string_scanner 1.1.0 [charcode source_span]
- syncfusion_flutter_core 19.2.49 [vector_math flutter]
- synchronized 3.0.0
- term_glyph 1.2.0
- test_api 0.4.2 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- timezone 0.7.0 [path]
- timing 1.0.0 [json_annotation]
- typed_data 1.3.0 [collection]
- universal_io 2.0.4 [collection crypto meta typed_data]
- url_launcher_linux 2.0.1 [flutter]
- url_launcher_macos 2.0.1 [flutter]
- url_launcher_platform_interface 2.0.4 [flutter plugin_platform_interface]
- url_launcher_web 2.0.2 [flutter flutter_web_plugins meta url_launcher_platform_interface]
- url_launcher_windows 2.0.1 [flutter]
- vector_math 2.1.0
- watcher 1.0.0 [async path pedantic]
- web_socket_channel 2.1.0 [async crypto stream_channel]
- win32 2.2.5 [ffi]
- xdg_directories 0.2.0 [meta path process]
- xml 5.1.2 [collection meta petitparser]
- yaml 3.1.0 [collection source_span string_scanner]

Same, thanks for reporting already

Same here, after upgrading Flutter

Hi @AgungLaksana
This is a duplicate of https://github.com/FirebaseExtended/flutterfire/issues/6770
Closing it as such
Thank you


Removed duplicate method call.

I removed a duplicate call I noticed while going through the code.  To test, I setup the necessary Azure resources for EventHub and ran all EventHub tests.  Screenshot of passing tests included.

Pretty obvious, thanks. I'll pull this in when I get around to it.

Merged locally. thanks!


fix(devkit): FileReplacement type typo

fixes #11294

Still can't get npm run test-large to pass on original master.

Superseded by https://github.com/angular/angular-cli/pull/12053


fix possibly negative index

deal with situations that index overflows and returns negative value. this leads to IndexOutOfBoundException

Pull Request Test Coverage Report for Build 3987

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 49.316%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

bundle/src/main/java/com/adobe/acs/commons/functions/RoundRobin.java
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

bundle/src/main/java/com/adobe/acs/commons/mcp/impl/ProcessErrorReportExcelServlet.java
1
66.22%

bundle/src/main/java/com/adobe/acs/commons/functions/RoundRobin.java
1
0.0%

Totals

Change from base Build 3971:
-0.01%

Covered Lines:
11867

Relevant Lines:
24063

💛  - Coveralls

@Nov11  - Could you add a Unit test for this? as well as a changelog entry?

Unit Tests and an entry in change log have bean added.


rename the CRD files to match the controller-gen naming convention

This patch is in preparation to switchover to the controller-gen for
generating CRD files from Go marker code. More importantly it makes it
easier to review the patch.
Related to #204

I think the compliance CRDs should be moved back to the crd folder, since I do not think we are able to control the CRD generation to that level of detail. Or do you have another plan for this @padlar? This means (partly) reverting https://github.com/aquasecurity/trivy-operator/pull/109. CC @josedonizetti

I think the compliance CRDs should be moved back to the crd folder (with the generate-names), since I do not think we are able to control the CRD generation to that level of detail. Or do you have another plan for this @padlar? This means (partly) reverting #109. CC @josedonizetti

this PR is about just renaming the CRD files. I was going to suggest moving the compliance CRDs back into the crd folder in the next PR as its not possible to generate the CRD files into 2 different folders automatically. Either manually move the compliance CRDs after generation or add a copy command post generation.

@padlar The compliance CRDs cannot be moved to crd folder at this point.
while suggesting auto generation for crds , need to see how to overcome custom validation on crds which we do not wish to lose. #232

LGTM 🚀


Patch-count resets

First many thanks for this nice tool.
The Patch-Count counts up (apprx. 1 per minute) til it reaches the correct amount of patches needed and somewhen resets to 0 and starts again counting up. Furthermore I get some curl errors in patch-manager _error.log (curl: (7) couldn't connect to host). We have no way to go around the proxy so I've set env_vars for proxy but still this error is there. also shouldn't there be the CVE-link beside the patch? or is this curl error responsible for that one?
Thanks in advance for your help.

The errors are caused by html/client/send_patches.php making a curl call to get package severity.  It only works for Ubuntu anyhow (right now), so if you aren't using ubuntu, replace with the following:
<?php
include '../lib/db_config.php';
$client_key = filter_input(INPUT_SERVER, 'HTTP_X_CLIENT_KEY');
$client_check_sql = "SELECT `id`,`server_name` FROM `servers` WHERE `client_key` = '$client_key' AND `trusted`=1 LIMIT 1;";
$link = mysql_connect(DB_HOST, DB_USER, DB_PASS);
mysql_select_db(DB_NAME, $link);
$client_check_res = mysql_query($client_check_sql);
if (mysql_num_rows($client_check_res) == 1) {
    $row = mysql_fetch_array($client_check_res);
    $server_name = $row['server_name'];
    $data = file_get_contents("php://input");
    mysql_query("DELETE FROM `patches` WHERE `server_name`='$server_name';");
    $package_array = explode("\n", $data);
    $suppression_sql = "SELECT * from `supressed` WHERE `server_name` IN('$server_name',0);";
    $suppression_res = mysql_query($sql);
    if (mysql_num_rows($suppression_res) == 0){
        $suppression_array = array("NO_SUPPRESSED_PACKAGES_FOUND");
    }
    else{
        while ($suppression_row = mysql_fetch_assoc($suppression_res)){
            $suppression_array[] = $suppression_row['package_name'];
        }
    }
    foreach ($package_array as $val) {
        $tmp_array = explode(":::", $val);
        $package_name = $tmp_array[0];
        $package_from = $tmp_array[1];
        $package_to = $tmp_array[2];
        $urgency = "unknown";
        if (!in_array($package_name, $suppression_array)) {
            $sql = "INSERT INTO patches(server_name,package_name,current,new,urgency,bug_url) VALUES('$server_name','$package_name','$package_from','$package_to','$urgency','$the_url');";
            mysql_query($sql);
        }
    }
}
mysql_close();

Also, patch_checker.sh should only run on each node once every 2 hours.  The only cron you should have on a node is to check-in every minute.
Let me know if this resolves your issue.

Created Gist of this fix:
https://gist.github.com/jonsjava/4fee9d36e99c2c61de67

Thanks for the fast fix :-) no more errors in error log.
will watch out if the patches counter changes.

counter still fine. everything working as expected now. thanks alot.


Add the "sending a simple message to a service" tutorial

Purpose
$title.

Overall, this is very lengthy to keep the focus and concentration of a user. Better to find ways to reduce the length such as adding similar instructions in a common generic page, etc.

Overall, this is very lengthy to keep the focus and concentration of a user. Better to find ways to reduce the length such as adding similar instructions in a common generic page, etc.

I don't think it is that lengthy. These are meant to be standalone tutorials. Since we are planning on maintaining the same format, someone who has followed another tutorial will know to skip to the section they need. There isn't much generic content anyway, other than setting up Ballerina.
Inspired by WSO2 tutorials - https://apim.docs.wso2.com/en/latest/tutorials/integration-tutorials/sending-a-simple-message-to-a-service/.


Prepare .csproj for code signing

Yet another pull request to prepare us to sign the .dlls. Also cleanup legacy code from Parse/Properties/AssemblyInfo.Portable.cs

Awesome. This will help us fix #46


Cannot Use DLL After Compiling

I am unable to use the cheat, "ShowLuaConsole" when I copy the resulting DLL from my compilation into Simcity 4. The cheat works if I just place the already compiled release into the plugins folder, so it's something I am doing wrong.
I have tried Visual Studios 2022 with Windows 10.0.22000, 10.0.17763.0, and 8.1 build tools with v143 and v141 toolsets. I am running Windows 11 Home if that makes a difference too.
Could I get some assistance in successfully compiling this DLL?

At the bottom of main.cpp, there's an RZGetComDllDirector function that the game uses to start the DLL. Right now, it's attached to a dummy director that doesn't do anything.
Take that function, move it to cGZExtraCheatsDllDirector.cpp, and change this line:
static cGZCustomDllDirector sDirector;

to:
static cSC4ExtraCheatsPluginCOMDirector sDirector;

This isn't documented anywhere so that's my fault.

Good Evening,
I made the above changes, but it still didn't work. I have forked this repo from master just to show my work: https://github.com/Songg45/gzcom-dll-test/commit/c1e5cd6a03bf11d97a3b7146c7578e06bc0b8dc5
Strangely enough, I can compile PoC: Message Targets's code as-is and that works fine.
My project properties:

I see your change is making changes to the x64 configuration. Make sure you're building for Win32. I just pushed a change to remove the x64 configurations since the game can't load 64-bit DLLs.

Excellent! I switched it to Win32 and I can get the Extra Extra cheats to work.
However, I still cannot get, "ShowLuaConsole" or the IGLE to work even using Extra Cheats. Is that code in one of the DllDirector files?

The in-game UIs are not part of a DLL director, they are scripted windows that the game spawns with the cIGZUIScriptService. There is already a release that add commands to spawn the following development UIs: ShowLuaConsole, ShowLotConfigChooser, ShowNetworkLotChooser, ShowSimulatorControl, and ShowExemplarDebugger.


Show tooltip for the whole item and not only for page title

What do you think about moving title attribute from node with page title to the container node? In this case the tooltip will appear not only for page title, but for page icon, url and the whole item itself.

Indeed, that's the expected behavior.


Enables e-mail batching in the scheduler

Description
Enables sending periodic batch e-mails from the scheduler rather than sending immediate failure/disable/scheduling failure e-mails on every event.
Motivation and Context
With a large pipeline, it's fairly easy to get hundreds of e-mails an hour when something goes wrong. This translates to thousands of e-mails per day. While this is a great resource for tracking down error messages of a particular task in your e-mail history, it's not nearly as great for understanding what the current issues are in your pipeline.
In order to make it more manageable, this groups together failures and shows sample error messages. Failures are ordered by how often they occur and it's easier to see from e-mail subject lines how many failures are occurring over time. Failure message history is preserved enough for most uses by including snippets for each task.
Have you tested this? If so, how?
I have included unit tests for all the pieces of this and have been running it for the last week or so, reducing my luigi e-mail volume by two orders of magnitude and making it easier to find and fix issues, also reducing the error volume. So the feature and its usefulness have both survived testing.

seems like a great feature!

Yeah, it makes identifying and fixing errors a lot easier.

Wow, this is a huge patch. I'll try to get to reviewing it soon :)

I tried to keep it small, but a lot of features are required to get a handle on the amount of e-mail I was getting.

@Tarrasch moving to luigi/server/batch_notifier.py would be a bit problematic because luigi.server would become ambiguous, referring to both luigi/server.py and luigi/server/. Perhaps we can move both in a separate PR?

Sure, we can do the luigi/server-subfolder in a separate PR. :)

I see you approved it. Should I rebase down to fewer commits for the merge?

@daveFNbuck, I merged this with every commit because the first 4 commits were very good. The last few commits were pretty crude but it's probably not worth anyone's time to meld them back into the right commits.


[TwitterV2Bridge] "Without Retweets" option hides Quote Tweets

The new TwitterV2Bridge doesn't show Quote Tweets if the "Without Retweets" option is checked.
The same thing doesn't happen with the original TwitterBridge.

I'll see what I can do. A quick check appears to show that quote tweets are retweets as far as the data returned by the Twitter v2 API is concerned, but it's possible there's a different way to parse/interpret the data.

Ok, so it looks like I was incorrectly treating quote tweets as retweets. That was a really easy fix. Next up is to try embedding the quoted tweet (as per issue #2697), which will take a bit more work.


Add warning when using load_folder on web

Objective
Help users who are using load_folder in wasm builds to find a slightly shorter path to figuring out why their stuff is broken.
Solution
Adds a warning to read_directory in the WasmAssetIo.
This is extremely similar to the warning already emitted a few lines below for watch_for_changes.

bors r+


"Oboga" is listed in the drop-down menu

Description:
When user writes "Oboga" the location appears in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Oboga"
Expected results
The location appears in the drop-down menu as "Comuna Oboga, OLT"

Step 3
Write in the search bar "Oboga"

Expected results
The location appears in the drop-down menu as "Sat Oboga, ** OLT** (Oboga)".


Fix task log appender will not be closed immediate after throwing exception

Purpose of the pull request
When logic task execute failed with exception, the log appended will not be closed immediate, it will be drop after 30 minutes, this is too long, we should close it immediate.
Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #14989 (e33259e) into dev (c5bc535) will decrease coverage by 0.02%.
 Consider uploading reports for the commit f503481 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14989      +/-   ##
============================================
- Coverage     38.90%   38.88%   -0.02%
+ Complexity     4606     4604       -2
============================================
  Files          1235     1235
  Lines         43425    43392      -33
  Branches       4805     4788      -17
============================================
- Hits          16894    16875      -19
+ Misses        24657    24644      -13
+ Partials       1874     1873       -1

Files
Coverage Δ

...ster/runner/execute/MasterTaskExecuteRunnable.java
0.00% <0.00%> (ø)

... and 25 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Create es-translation.json

Hi, I created the file with the Spanish translation, I hope it will be of some help.

¡Muchas Gracias!
Would you like to be in the credits? I really appreciate your help. Just let me know what name you would like me to use.
Also, you can check the translations here to make sure they look right in context.

Hi, don't worry about it. I don't need to be in the credits.
About the translation, I was doing local tests yesterday and it looks fine to me.


array concatenation

Simple submission for array concatenation task to test contribution process

Hi! :)
Thanks for this contribution!

np!
Do I copy this solution to the Rosetta wiki?

@ibnopcit Please!


sst deploy hangs forever in ci

I deployed a new app with sst ion from my local machine, it deployed fine, however now that I'm trying to get it deploying through ci (github actions) I've run into a bit of an issue. The app deploys successfully (I can see my changes), however the command hangs forever and never actually exits. I enabled verbose logging and can see that it gets to "unlocking", but never progresses past that point (without verbose logging, nothing is ever printed). Full logs included below.
I followed the old docs for setting up OIDC & creating the role (thus have used a GitHub role with AdministratorAccess for now), and given the deployment is being updated it would seem that's working fine.
Additionally I tried running sst unlock --stage=sandbox --verbose in ci, and that works fine, it's just the deploy command that hangs.
Here's the action:
name: deploy-sandbox
on:
  push:
    branches:
      - main
concurrency:
  group: ${{ github.workflow }}
jobs:
  deploy-sandbox:
    name: Deploy Sandbox
    runs-on: ubuntu-latest
    permissions:
      # required for aws-actions/configure-aws-credentials
      id-token: write
      # required for actions/checkout
      contents: read
    steps:
      - uses: actions/checkout@v4

      # setup node
      - name: Setup node
        uses: actions/setup-node@v4
        with:
          node-version: 20
      - run: npm ci

      # install sst
      - name: Install SST
        run: curl -fsSL https://ion.sst.dev/install | bash

      # configure aws credentials
      - name: Configure AWS credentials
        uses: aws-actions/configure-aws-credentials@v4
        with:
          role-to-assume: arn:aws:iam::<redacted>:role/GitHub
          aws-region: us-east-1

      # TODO: debug
      # - name: DEBUG
      #   run: sst unlock --stage=sandbox --verbose

      # deploy
      - name: Deploy
        run: sst deploy --stage=sandbox --verbose
        # TODO: DEBUG
        timeout-minutes: 30

the sst config:
/// <reference path="./.sst/platform/config.d.ts" />

export default $config({
    app(input) {
        return {
            name: "core-app",
            removal: input?.stage === "production" ? "retain" : "remove",
            home: "aws",
        };
    },
    async run() {
        new sst.aws.Nextjs("<redacted>");
    },
});

and the full deploy logs sst deploy --stage=sandbox --verbose:
time=2024-06-25T14:00:42.726Z level=INFO msg="checking for pulumi" path=/home/runner/.config/sst/bin/pulumi
time=2024-06-25T14:00:42.905Z level=INFO msg="checking for bun" path=/home/runner/.config/sst/bin/bun
time=2024-06-25T14:00:42.909Z level=INFO msg="initializing project" version=0.0.441
time=2024-06-25T14:00:42.9[10](https://github.com/<redacted>/core-app/actions/runs/9663868285/job/26657142993#step:8:11)Z level=INFO msg="esbuild building"
time=2024-06-25T14:00:42.912Z level=INFO msg="esbuild built" outfile=/home/runner/work/core-app/core-app/.sst/eval/eval-1719324042910.mjs
time=2024-06-25T14:00:42.912Z level=INFO msg="evaluating config"
time=2024-06-25T14:00:42.945Z level=INFO msg="config evaluated"
time=2024-06-25T14:00:42.945Z level=INFO msg="checking platform"
time=2024-06-25T[14](https://github.com/<redacted>/core-app/actions/runs/9663868285/job/26657142993#step:8:15):00:42.945Z level=INFO msg="checking provider" name=aws version=latest compare=latest
time=2024-06-25T14:00:42.946Z level=INFO msg="credentials found"
time=2024-06-25T14:00:42.946Z level=INFO msg="fetching bootstrap"
time=2024-06-25T14:00:43.269Z level=INFO msg="found existing bootstrap" data="{\"version\":1,\"asset\":\"sst-asset-<redacted>\",\"state\":\"sst-state-<redacted>\"}"
time=2024-06-25T14:00:43.270Z level=INFO msg="loaded config" app=core-app stage=sandbox
time=2024-06-25T14:00:43.270Z level=INFO msg="running stack command" cmd=deploy
time=2024-06-25T14:00:43.270Z level=INFO msg="INFO locking app=core-app stage=sandbox"
time=2024-06-25T14:00:43.270Z level=INFO msg="INFO getting data key=lock app=core-app stage=sandbox"
time=2024-06-25T14:00:43.521Z level=INFO msg="INFO putting data key=lock app=core-app stage=sandbox"
time=2024-06-25T14:00:43.772Z level=INFO msg="INFO pulling state app=core-app stage=sandbox out=/home/runner/work/core-app/core-app/.sst/.pulumi/stacks/core-app/sandbox.json"
time=2024-06-25T14:00:44.425Z level=INFO msg="INFO getting passphrase app=core-app stage=sandbox"
time=2024-06-25T14:00:44.764Z level=INFO msg="INFO getting data key=secret app=core-app stage=sandbox"
time=2024-06-25T14:00:45.022Z level=INFO msg="INFO getting passphrase app=core-app stage=sandbox"
time=2024-06-25T14:00:45.346Z level=INFO msg="esbuild building"
time=2024-06-25T14:00:45.443Z level=INFO msg="esbuild built" outfile=/home/runner/work/core-app/core-app/.sst/platform/eval/eval-[17](https://github.com/<redacted>/core-app/actions/runs/9663868285/job/26657142993#step:8:18)19324045346.mjs
time=2024-06-25T14:00:45.443Z level=INFO msg="tracked files"
time=2024-06-25T14:00:45.586Z level=INFO msg="built workspace"
time=2024-06-25T14:00:45.762Z level=INFO msg="built stack"
time=2024-06-25T14:00:45.901Z level=INFO msg="built config"
time=[20](https://github.com/<redacted>/core-app/actions/runs/9663868285/job/26657142993#step:8:21)24-06-25T14:00:45.901Z level=INFO msg="running stack command" cmd=deploy
time=2024-06-25T14:02:00.562Z level=INFO msg="done running stack command"
time=2024-06-25T14:02:00.562Z level=INFO msg="INFO putting summary app=core-app stage=sandbox"
time=2024-06-25T14:02:00.562Z level=INFO msg="INFO putting data key=summary app=core-app stage=sandbox/ggyc99ffkpmiwk5kpimg17jy"
time=2024-06-25T14:02:00.846Z level=INFO msg="stack command complete"
time=2024-06-25T14:02:01.[22](https://github.com/<redacted>/core-app/actions/runs/9663868285/job/26657142993#step:8:23)5Z level=INFO msg="INFO putting links app=core-app stage=sandbox"
time=2024-06-25T14:02:01.225Z level=INFO msg="INFO putting data key=link app=core-app stage=sandbox"
time=20[24](https://github.com/<redacted>/core-app/actions/runs/9663868285/job/26657142993#step:8:25)-06-25T14:02:01.225Z level=INFO msg="INFO getting passphrase app=core-app stage=sandbox"
time=2024-06-[25](https://github.com/<redacted>/core-app/actions/runs/9663868285/job/26657142993#step:8:26)T14:02:01.861Z level=INFO msg="INFO pushing state app=core-app stage=sandbox from=/home/runner/work/core-app/core-app/.sst/.pulumi/stacks/core-app/sandbox.json"
time=2024-06-25T14:02:02.577Z level=INFO msg="INFO unlocking app=core-app stage=sandbox"
Error: The action 'Deploy' has timed out after 30 minutes.

@notsoluckycharm , I had sst stuck for me as well, and setting asyncpreemptoff=1 released the blockage somehow.
How does it affect on sst ?


sub-workflow defaults not overridden by provided value

Hello, I noticed in testing dxCompiler 2.4.10 that sub-workflow default input values are not overridden by a different provided value.
For example with the following 2 WDL files
workflow1.wdl
import "workflow2.wdl"

workflow workflow1 {
    input {
    ...
    }
   call workflow2 {
       input:
            my_input=["hello"]
   }
...
}

workflow2.wdl
workflow workflow2 {
    input {
        Array[String] my_input = []
    }
...
}

when workflow2 is run, it is using [] rather than ["hello"]

For reference: https://github.com/openwdl/wdl/blob/main/versions/1.1/SPEC.md#task-inputs:
A task's input section declares its input parameters. The values for declarations within the input section may be specified by the caller of the task. An input declaration may be initialized to a default value or expression that will be used when the caller does not specify a value

so when workflow1 invokes workflow2, workflow2 should execute with my_input = ["hello"].
@commandlinegirl let's see if 2.10.1 has the correct behavior

I tested this with dxCompiler 2.10.1 and observe the correct behavior (I extended our test for subworkflow defaults with arrays, to be sure, here. If you see an issue again, please ping us, thank you!

Hmm, I just tested it additionally with dxCompiler 2.4.10 (using this updated test) and cannot reproduce this issue.

I verified that the issue was fixed in dxCompiler-2.7.0 and updated the release notes for that release. The input values before were indeed not used inside the subworkflow, but starting from 2.7.0 the behavior is correct. cc @emiloslavsky. Our integration test is updated to cover that.

great thanks @commandlinegirl and @emiloslavsky !


zzPLAINzz and zzXSSzz not getting converted

Google account sign-up flow not triggering. It's a POST request with the zzXSSzz in the body.

I'm too facing same problem.. not able to trigger even a simple xss on demo.testfire.net or xss-game.appspot.com/level1

hmm, @sohelahmed786  how are you trying to trigger the flow? I'm curious if it is a related issue or a new one. Here's what I get when I try it:

which seems to be working as expected.

in my browser it's not working as expected.. don't know why..

This might be a silly question, but you clicked the Search button right?

ofcourse i clicked , without it how will it execute my payload... Comparing to your image mine doesn't has url or method name... that means it has some problem maybe because of some configuration in my firefox..

lol it's working on chrome ... what happened to my firefox...

Yea that was why I asked if you clicked the submit button. Tracy should catch the HTTP request like it did in my example. Do you have other extensions installed on Firefox? Or an HTTP proxy setup? Maybe they aren't playing well together.

hmm maybe it's becz of firefox version. I'm using 60.8.0esr (64-bit). I tried disabling all addons except tracy... problem persists.

I should also mention that this issue was written up because I was having issues with the hnxgoyqccn and \"'<cxzbkixdeh> payloads, which only get converted when the network requests get triggered. It looks like from your screenshot, you are using the \"'<nnmlvnctwr> payloads, which get generated right then and there.

hmm maybe it's becz of firefox version. I'm using 60.8.0esr (64-bit). I tried disabling all addons except tracy... problem persists.

OOOOO, yea that is an old old Firefox version. You can probably see the errors if you open the dev console in the page and or debug the add-on and open the background page console. You'll need update to get that working.

I could probably do a better job making the extension more backward compatible, I think I use some newer APIs that aren't in that version of Firefox, such as [].flat() https://caniuse.com/#search=flat

👍 sure thing! happy to help


Serialization error when importing from MongoDB

Description
I tried to import documents with meilisync from the MongoDB free plan: https://cloud.mongodb.com/ and got a serialization error:
TypeError: Object of type ObjectId is not JSON serializable
Data format:
I used the sample data proposed by MongoDB which looked like:
{
  "_id": { "$oid": "64c91330c2a74205feef694b" },
  "account_id": { "$numberInt": "278603" },
  "limit": { "$numberInt": "10000" },
  "products": ["Commodity", "InvestmentStock"]
}

Configuration file
debug: true
plugins:
  - meilisync.plugin.Plugin
progress:
  type: file
source:
  type: mongo
  host: mongodb+srv://cluster_IP.mongodb.net
  port: 27017
  username: userName
  password: passWord
  database: sample_analytics
meilisearch:
  api_url: http://localhost:7700/
  api_key: 'masterKey'
  insert_size: 100
  insert_interval: 10
sync:
  - table: accounts
    index: accounts
    full: true

Screenshots & Logs:

I put my logs in a file just in case:
output_error.txt
Note:
I tried to remove full: true by:
    fields:
      account_id:
      limit:
      products:

But I received another error that I will describe in another issue.

Hi @alallema :wave: . Did you prepare the data from Mongo to send or did Meilisync do it itself (sorry, I'm only familiar with meilisync at a very high level). The issue here is the ObjectId needs to be converted to a string before it can be serialized to JSON. I can make some suggestions how to do that once I know where the data came from.

Hi @sanders41,
Thank you for replying 😊. No, I haven't prepared anything. I've used the samples proposed by MongoDb to populate my database, and it seems that all _id fields are created in this format by default on MongoDb Cloud.
I've also created a collection using the movie dataset and I've got another error described here #17


Add random spooky gif feature.

Addresses issue #8, adding random spooky gif feature that takes use of the GIPHY API.
To take use of this cog, create a env. variable named GIPHY_TOKEN and use the token you get from https://developers.giphy.com, they have a very simple process of creating an app, just create an account and you should be able to make your app and get the API key.

Alright, resolved the requested changes. Feel free to test.

Any progress on the requested changes?

@sco1 Sorry, I'm currently on vacation and have no PC access. Will get this done soon.


golangci-lint linter: automatically infer the execution directory

background:
😱🚀 Linter: golangci-lint
Repo: goplus/builder
PR:   https://github.com/goplus/builder/pull/615
Event: d4605c30-3469-11ef-83a5-c58febdb5641
Unexpected: level=error msg="[linters_context] typechecking error: pattern ./...: directory prefix . does not contain main module or its selected dependencies"

In some monorepos, Go code is not placed in the main directory, and the current golangci-lint cannot automatically discover it.
Reviewbot currently supports custom workDir, but it would be better if it could be automatically determined.

FYI: https://github.com/golangci/golangci-lint/issues/825

fixed by #235


Add files via upload

@menonkrishna57 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@menonkrishna57 You might miss my previous comment. Please react.

I got access via GitHub. If you still need it though the website is https://engineering.nmims.edu

@menonkrishna57 No, we don't insist on adding the school to the trusted list, especially if the domain has been changed. I close this request without merging.
If you will need to update the info or repeat the request, please make a new one. Please make a comment with the requested information in a new request, if you decide to create it.


Correct target.type from AverageUtilization to Utilization

The HorizontalPodAutoscaler target.type must be either Utilization, Value, or AverageValue.
Correct the target.type from AverageUtilization to Utilization.
Signed-off-by: JenTing Hsiao<EMAIL_ADDRESS>

/check-cla

@jenting do you any problem while signing the cla?

@jenting do you any problem while signing the cla?

I signed the corporation CLA half an hour ago. But it seems like the CLA check does not pass, any clue?

@jenting On the Linux Foundation Page Is it showing your_company_name CLA ?

/check-cla

@jenting On the Linux Foundation Page Is it showing your_company_name CLA ?

Nope! Just link my Github account to Linux Foundation Page, and now it shows up, hope it will working :pray:

/assign @gochist

/unassign

/assign @jenting

/unassign

Can I get a technical /lgtm?
/cc @kubernetes/sig-auth-pr-reviews
Thanks!

/remove-sig auth
/sig autoscaling

/cc @kubernetes/sig-autoscaling-pr-reviews

@kubernetes/sig-autoscaling-pr-reviews Ping

/lgtm from sig-autoscaling

/lgtm
/approve

/assign @jaredbhatti


Speed up CI by re-using relayer binary for E2E tests

We could likely significantly speed up the end-to-end test job by copying the relayer binary built as part of the Rust job into the image instead of rebuilding it.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

The original CI infrastructure was done as part of https://github.com/informalsystems/ibc-rs/pull/544.
This was later extended with comprehensive tests in https://github.com/informalsystems/ibc-rs/pull/571.
The work in the original CI consists of three docker images:

two of them were gaia chain images called "informaldev/ibc-0" and "informaldev/ibc-1"
the third image is for the relayer, and is dependent on the gaia images; this is built on-the-fly upon every commit to a pull-request

There are two basic differences between the original work and the new PR:

the relayer binary is now compiled as part of the Github action, and then this binary is made available (cp) to the relayer docker image:

https://github.com/informalsystems/ibc-rs/blob/81af0b0b0418c4a365617e3cbf6006db80a3c831/.github/workflows/e2e.yaml#L13-L17
in the original work, the binary was compiled directly inside the docker-compose task:
https://github.com/informalsystems/ibc-rs/blob/148e196308a217542f2dc49bfe5881362bf1013d/ci/relayer.Dockerfile#L28

the cargo build command for obtaining the relayer binary no longer relies on --all --release, shortening the build cycle by ~5 minutes (almost halved)


Use remote Docker Engine via SSH or using Docker context

Steps to reproduce the issue:

Create the DockerClient that use connection via SSH

using Docker.DotNet;
DockerClient client = new DockerClientConfiguration(
    new Uri("ssh://user@host.com"))
     .CreateClient();

What actually happened?:
I've got an excetption:
Unhandled exception. System.Exception: Unknown URL scheme ssh
   at Docker.DotNet.DockerClient..ctor(DockerClientConfiguration configuration,
Version requestedApiVersion) in D:\work\Docker.DotNet\src\Docker.DotNet\DockerCl
ient.cs:line 135
   at Docker.DotNet.DockerClientConfiguration.CreateClient(Version requestedApiV
ersion) in D:\work\Docker.DotNet\src\Docker.DotNet\DockerClientConfiguration.cs:
line 54
   at Docker.DotNet.DockerClientConfiguration.CreateClient() in D:\work\Docker.D
otNet\src\Docker.DotNet\DockerClientConfiguration.cs:line 49
   at DockerTester.Program.Main(String[] args) in C:\Users\nagorski\RiderProject
s\DockerTester\DockerTester\Program.cs:line 21
   at DockerTester.Program.<Main>(String[] args)

What did you expect to happen?:
I should be able to use remote Docker via SSH. Currently Docker CLI has support for Docker context. I think I should be able to use Docker.DotNet API in the same way.
For Example, I can add Docker context using the following command:
docker context create --docker<EMAIL_ADDRESS>--description="Remote engine" my-remote-engine

Then I can use the remote Docker engine using the --context parameter:
docker --context my-remote-engine images

More information:

https://docs.docker.com/engine/security/protect-access/#use-ssh-to-protect-the-docker-daemon-socket
https://docs.docker.com/engine/context/working-with-contexts/

Currently I can't use either Docker Engine via SSH or docker context.
Additional information:
Output of dotnet --info:
❯ dotnet --info
.NET SDK (reflecting any global.json):
 Version:   6.0.100-rc.1.21463.6
 Commit:    e627d556a1

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.19042
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\6.0.100-rc.1.21463.6\

Host (useful for support):
  Version: 6.0.0-rc.1.21451.13
  Commit:  d7619cd4b1

.NET SDKs installed:
  2.1.520 [C:\Program Files\dotnet\sdk]
  3.1.300 [C:\Program Files\dotnet\sdk]
  5.0.204 [C:\Program Files\dotnet\sdk]
  5.0.401 [C:\Program Files\dotnet\sdk]
  6.0.100-rc.1.21463.6 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.All 2.1.24 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.24 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.19 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 5.0.10 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.0-rc.1.21452.15 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.24 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.19 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.10 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.0-rc.1.21451.13 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 3.1.4 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 3.1.19 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 5.0.7 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 5.0.10 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.0-rc.1.21451.3 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET runtimes or SDKs:
  https://aka.ms/dotnet-download

What version of Docker.DotNet?:
latest

@galvesribeiro do you have plan to fix add this ability? connect to docker with ssh

@behnam-basketasia I assume PR are most welcome ;-)

There is already a fork TrapTech.Docker.DotNet.SSH which is providing the function you are asking for. Maybe that fork can merged into here.
I personally would prefer that the ssh support is build into this lib here.
My use case:
I'm not exposing docker api on remote hosts for security reasons. I also don't install any docker stuff on my windows computer. This is the reason why I rely on the the SSH. I also use the test containers linked in a comment before.


[decomp] Add tool to print type information from a memory dump

Pull Request Test Coverage Report for Build 858739688

0 of 166 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 69.034%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tools/MemoryDumpTool/main.cpp
0
166
0.0%

Totals

Change from base Build 855307206:
-0.2%

Covered Lines:
34569

Relevant Lines:
50075

💛  - Coveralls

codacy failure is a false positive. The check is here:
https://github.com/david-a-wheeler/flawfinder/blob/293ca17d8212905c7788aca1df7837d4716bd456/flawfinder#L1057 which mistakenly recognizes me defining a template method called read as the POSIX read function being called.


support dllimport data

struct Game
{
	unsigned short a[3];
};
struct Game game;

int main()
{
	return game.a[0];
}

This simple program compiled with mingw can be debugged in VS thx to cv2pdb.
But if the data is moved to a dll (__declspec(dllimport) struct Game game;) VS does not recognize any game variable anymore inside main.
The main difference in main.o DWARF debug info seems to be
DW_TAG_variable
  DW_AT_name	("game")
  DW_AT_type	(0x0000009a "Game")
  DW_AT_external	(true)
- DW_AT_location	(DW_OP_addr 0x0)
+ DW_AT_declaration	(true)

I don't see this working when compiling dll and exe with the Microsoft compiler, so it's probably not so easy to implement. You would have to make the entry in the import table a reference to a Game instance and rename it "game" somehow...

Seems like my edit got lost. When compiling with msvc the pdb contains
S_GDATA32 `_imp__game`
           type = 0x1002 (Game), addr = 0002:0264
S_PUB32 `__imp__game`
           flags = none, addr = 0002:0264

The Game type definition itself is also present in the cv2pdb produced pdb.

Indeed, the symbol is there, but it needs to be extern "C" to actually be able to evaluate it in the watch window because it needs to be spelled out as is (game doesn't work for me). Additionally it is typed badly, because the value in the import table is a pointer to Game, not the value itself.
Maybe the DWARF output makes more sense, though.

I did compile it as C. Indeed game is not recognized anymore when compiling with msvc. Odd, I'm pretty sure it worked before.
del /Q *.pdb
cl -nologo -O1 -Z7 -LD dll.c -link -OPT:REF
cl -nologo -O1 -Z7 main.c dll.lib -link -OPT:REF

There's also a name and type difference between x86 and x64: void* __imp__game vs. Game __imp_game.

There's also a name and type difference between x86 and x64: void* __imp__game vs. Game __imp_game.

That's the usual underscore prepended to the game symbol on x86.
The latest commit https://github.com/rainers/cv2pdb/commit/243f8daa3289f205bad5dd73e425fea9c6761236 adds support for C like symbols, i.e.e that don't have additional mangling.

That fixes the sample code 👍
But in the real code it gets a null address for whatever reason:  | Game &

IIRC cv2pdb doesn't (fully) support DLLs that are relocated to an address other than the default address. If your "main" is actually in a DLL itself that might be the problem.
If not, a test case for reproduction might be useful.

No it isn't relocated. Looks like the COFF symbol table is wrong to begin with:
965C 0000162C SECT5  notype       External     | __imp__game
..
9866 00000000 UNDEF  notype       0x6a UNKNOWN SYMBOL CLASS | _game

Hence the pdb entries:
  S_GDATA32 [size = 24] `game`
  type = 0x1325 (Game&), addr = 0006:5676    =  0x162c

While the correct address is
.idata:0078062C                 extrn game:dword
.idata begin: 0077FF40 => offset 6EC = 1772d

Why would that matter? The pointer to the data is inside the .exe.
One (probably unrelated) problem in PEImage::findSymbol is it doesn't handle section number 0: https://docs.microsoft.com/en-us/windows/win32/debug/pe-format#section-number-values. See this dumpbin /symbols excerpt, but this affects only some variables depending on the entry order:
9383 00000000 UNDEF  notype       0x6a UNKNOWN SYMBOL CLASS | _split1at
...
9644 00001B04 SECT5  notype       External     | __imp__split1at

One unrelated problem in PEImage::findSymbol is it doesn't handle section number 0: https://docs.microsoft.com/en-us/windows/win32/debug/pe-format#section-number-values. See this dumpbin /symbols excerpt, but this affects only some variables depending on the entry order:
9383 00000000 UNDEF  notype       0x6a UNKNOWN SYMBOL CLASS | _split1at
...
9644 00001B04 SECT5  notype       External     | __imp__split1at

diff --git a/src/PEImage.cpp b/src/PEImage.cpp
index 1f916c2..6121a35 100644
--- a/src/PEImage.cpp
+++ b/src/PEImage.cpp
@@ -809,10 +809,11 @@ int PEImage::findSymbol(const char* name, unsigned long& off, bool& dllimport) c
 	{
 		IMAGE_SYMBOL* sym = (IMAGE_SYMBOL*) (symtable + i * sizeof_sym);
 		const char* symname = sym->N.Name.Short == 0 ? strtable + sym->N.Name.Long : (char*)sym->N.ShortName;
-		if(symbolMatches(name, symname, dllimport))
+		auto section = bigobj ? ((IMAGE_SYMBOL_EX*)sym)->SectionNumber : sym->SectionNumber;
+		if(section > 0 && symbolMatches(name, symname, dllimport))
 		{
 			off = sym->Value;
-			return bigobj ? ((IMAGE_SYMBOL_EX*)sym)->SectionNumber : sym->SectionNumber;
+			return section;
 		}
 		i += sym->NumberOfAuxSymbols;
 	}

Indeed the dll got relocated. But why would that matter? The pointer to the data is in the exe .idata section.

I meant the binary you are debugging. If that is an exe, there is no relocation and that's ok then.

One unrelated problem in PEImage::findSymbol is it doesn't handle section number 0:

I don't think these can appear in the executable. External symbols are only in object files and must be resolved by the linker.

Well see above, mingw did produce such entries in the coff symbol table.
I really don't know where that offset is coming from. IDiaSymbol gives
getVirtualAddress(): 0x00000000001b3b74 => should be 0x36BB74
getAddressSection(): 0x00000006 => seg + 1
getAddressOffset(): 0x00001b74 => correct

If you provide an example executable, I can have a look.

Well see above, mingw did produce such entries in the coff symbol table.

I guess you are referring to the object file.
BTW: The executable I tried with
g++.exe -g maingame.cpp game.lib (g++.exe (i686-posix-dwarf-rev0, Built by MinGW-W64 project) 8.1.0) isn't considered correct by dumpbin:
>dumpbin /symbols a.exe
Microsoft (R) COFF/PE Dumper Version 14.24.28316.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file a.exe

File Type: EXECUTABLE IMAGE
LINK : fatal error LNK1328: missing string table

No I do mean the exe as that's where cv2pdb reads the symbol address from. Mine is built with Linux MinGW (via WSL) and dumpbin works.

Your DLL still causes the same failure with dumpbin for me (from VS 2019 16.4.4). If I dump it with objdump -x, you can find these lines:
There is an import table in .idata at 0x76a000

The Import Tables (interpreted .idata section contents)
 vma:            Hint    Time      Forward  DLL       First
                 Table   Stamp     Chain    Name      Thunk
 0036a000	0036a0dc 00000000 00000000 00370df0 0036af40

	DLL Name: IMAGEHLP.DLL
...
 0036a064	0036a364 00000000 00000000 00371b68 0036b1c8

	DLL Name: keeperfx.dll
	vma:  Hint/Ord Member-Name Bound-To
	36c5f8	    1  _DK_Border
...
	36e114	  945  _DK_game
...
Sections:
Idx Name          Size      VMA       LMA       File off  Algn
  4 .idata        00007c04  0076a000  0076a000  001a8800  2**2
                  CONTENTS, ALLOC, LOAD, DATA
...
SYMBOL TABLE:
[  0](sec -2)(fl 0x00)(ty   0)(scl 103) (nx 1) 0x0000002b crtexe.c
...
[38492](sec  5)(fl 0x00)(ty   0)(scl   2) (nx 0) 0x0000162c __imp___DK_game
...
[39014](sec  0)(fl 0x00)(ty   0)(scl 106) (nx 0) 0x00000000 __DK_game

Observations:

945*6+6 == 0x162c. I think the symbol information is wrong as it contains the offset in the import data for keeperfx.dll, but not in the .idata section.
segment 6 in the debug info seems questionable, it should rather be 5?
the "external" symbol with sec 0 seems to be some kind of alias to the import symbol and should rather be ignored.

My dumpbin is from the latest preview. Maybe they fixed something.
0x76b62c (the Game*) - 0x76a000 (.idata) is 162c. Seems correct.
And idk why the pdb has segment + 1 (https://github.com/rainers/cv2pdb/blob/243f8daa3289f205bad5dd73e425fea9c6761236/src/dwarf2pdb.cpp#L1616) but it worked in the minimal test case.

0x76b62c (the Game*) - 0x76a000 (.idata) is 162c. Seems correct.

Indeed, not sure what I was looking at.
The missing offset is 0x1c0000, which is the size of the .bss section before .idata. This section does not exist in the VC compiled executable (although there is a .textbss section as the first section).

DIA (read via llvm-pdbutil pretty --globals) gives 0x01b362c (offset from imagebase). 76b62c - (1b362c + 400000) = 1B8000 which is only approx. the size of bss: 1BF3F0
But in the minimal test case there's the same section order and there it works.

The expected pointer is at 0x76b62c. You have to take the aligned sections sizes, or just the differences of the virtual addresses.

But in the minimal test case there's the same section order and there it works.

I suspect it works because the sections are all of size 0x1000, and the current debug info is off by one section.

I suspect it works because the sections are all of size 0x1000, and the current debug info is off by one section.

Yes that must be it.

I think I found the reason, the section map seems to be wrong. The bss section entry is set to size 0.
That plus the off-by-one error explains the difference.

Cool, looks good. I've pushed the fix (the segment should be changed in findSymbol, though.


[Bug] Clicking on organization names does nothing

Description
Upon clicking on the name of the organization, nothing happens except for a few times. Looks like there needs to be a name for the scroll to work.
Also, adding a smooth scroll may be more user friendly.
Steps to reproduce the behavior:
Screencast: https://recordit.co/FS56YQfC80
Important Details

Version: Cloud
OS: Windows 10
Browser: Microsoft Edge

Duplicate of #807  different result but the root cause is the same


little refactoring with a switch between JRuby and MRI in Moped::Socket class

Little proposal of refactoring after commit : https://github.com/mongoid/moped/commit/5d66bc515edb4c24673a1536d8743ba0c2d34add
To me, this commit allow more readable the code than previous. No feature adding in this commit. Feedback accepted :)

Thanks! That does look better, though I don't think I'll pull it in. I'm going to be ripping out that part of the socket class soon; I've got a much better idea for how to handle connection failures that won't have the same performance penalties as currently exist.
Cheers.

No problem.